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.

Sign up for GitHub

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

Jump to bottom

Remove explicit constructor cppcheck suppression #230

Closed
macumber opened this issue Sep 9, 2020 · 1 comment · Fixed by #511
Closed

Remove explicit constructor cppcheck suppression #230

macumber opened this issue Sep 9, 2020 · 1 comment · Fixed by #511
Assignees
@j-lorrey
Labels
Developer Issue 🤓 Enhancement Request New feature or request
Milestone
OpenStudio Applic...

Comments

@macumber
Copy link
Collaborator

macumber commented Sep 9, 2020

https://github.com/openstudiocoalition/OpenStudioApplication/blob/develop/.github/workflows/cppcheck.yml
@macumber macumber mentioned this issue Sep 9, 2020
Merged
@jmarrec jmarrec mentioned this issue Oct 5, 2020
Merged
@macumber macumber added this to the OpenStudio Application 1.3.0 milestone May 29, 2021
j-lorrey added a commit to j-lorrey/OpenStudioApplication that referenced this issue Apr 29, 2022
@j-lorrey
[openstudiocoalition#230] Remove explicit constructor cppcheck suppre…
debd894 
…ssion
j-lorrey added a commit to j-lorrey/OpenStudioApplication that referenced this issue May 2, 2022
@j-lorrey
[openstudiocoalition#230] Address lines that were throwing noExplicit…
676a930 
…Constructor warnings
j-lorrey added a commit to j-lorrey/OpenStudioApplication that referenced this issue May 2, 2022
@j-lorrey
[openstudiocoalition#230] Address lines that were throwing noExplicit…
a4d88e7 
…Constructor warnings
jmarrec pushed a commit to j-lorrey/OpenStudioApplication that referenced this issue May 4, 2022
@j-lorrey @jmarrec
[openstudiocoalition#230] Remove explicit constructor cppcheck suppre…
ff36f26 
…ssion
jmarrec pushed a commit to j-lorrey/OpenStudioApplication that referenced this issue May 4, 2022
@j-lorrey @jmarrec
[openstudiocoalition#230] Address lines that were throwing noExplicit…
1d792dc 
…Constructor warnings
jmarrec pushed a commit to j-lorrey/OpenStudioApplication that referenced this issue May 5, 2022
@j-lorrey @jmarrec
[openstudiocoalition#230] Remove explicit constructor cppcheck suppre…
3cceac9 
…ssion
jmarrec pushed a commit to j-lorrey/OpenStudioApplication that referenced this issue May 5, 2022
@j-lorrey @jmarrec
[openstudiocoalition#230] Address lines that were throwing noExplicit…
3c0f130 
…Constructor warnings
jmarrec added a commit that referenced this issue May 5, 2022
@jmarrec
Merge pull request #511 from j-lorrey/230_cppcheck
600a96e 
[#230] Remove explicit constructor cppcheck suppression
@jmarrec jmarrec mentioned this issue May 9, 2022
Merged
@jmarrec
Copy link
Collaborator

jmarrec commented May 9, 2022

Fixed via #511

@jmarrec jmarrec closed this as completed May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@j-lorrey j-lorrey

Labels
Developer Issue 🤓 Enhancement Request New feature or request
Projects
None yet
Milestone
OpenStudio Application 1.4.0
Development

Successfully merging a pull request may close this issue.

[#230] Remove explicit constructor cppcheck suppression j-lorrey/OpenStudioApplication
3 participants
@macumber @jmarrec @j-lorrey