Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 740 (missing override method caused incorrect return type) #763

Merged
merged 3 commits into from
Feb 21, 2020

Conversation

lwhite1
Copy link
Collaborator

@lwhite1 lwhite1 commented Feb 21, 2020

Thanks for contributing.

Description

What was changed

Testing

Did you add a unit test?

BoxBuilder does not override name(string) as it should, and the return value of the inherited method is incorrect.
The codacy issues have been here forever, but are being flagged as new.
@lwhite1 lwhite1 merged commit 1cb6f40 into master Feb 21, 2020
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@lwhite1 lwhite1 deleted the issue_740 branch February 21, 2020 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant