-
Notifications
You must be signed in to change notification settings - Fork 1
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
METplus-Internal_39 SonarQube scanning #101
Conversation
done with 1.1.0 release update for next release
Fix reading of data for Python 3.8, branch Feature 85 fix read, Issue 85
Feature 84 docker centos7 - change from centos8
Feature 90 User has a database without a password
Issue #93 updates for beta1
update to dev for beta2
Feature 95 doc db to io
Feature change name
The Github actions test for compare two databases has been consistently failing prior to this work. I've asked Venita about this but I'm going to approve this PR because the sonarqube scanning is working. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Successfully ran the scan and viewed the dashboard. METdataio sonarqube passes. The Github Actions test: compare two databases has been failing on earlier pushes for other branches. I'm approving this because it appears that these changes are not responsible for the database tests failing.
Pull Request Testing
It's configured to scan the code nightly at seneca under met_test account by cron.
The scanning result is viewable by web browser at http://mandam:9090. The user name and password exist at sonar-project.properties
Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:
Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [No]
Do these changes include sufficient testing updates? [No]
Will this PR result in changes to the test suite? [No]
If yes, describe the new output and/or changes to the existing output:
Please complete this pull request review by [Fill in date].
Pull Request Checklist
See the METplus Workflow for details.
Select: Reviewer(s)
Select: Organization level software support Project or Repository level development cycle Project
Select: Milestone as the version that will include these changes