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

Add support for CodeQL and Sonarcloud for enhanced repo security #180

Open
stan-dot opened this issue Aug 7, 2024 · 5 comments
Open

Add support for CodeQL and Sonarcloud for enhanced repo security #180

stan-dot opened this issue Aug 7, 2024 · 5 comments
Assignees
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed

Comments

@stan-dot
Copy link
Contributor

stan-dot commented Aug 7, 2024

https://github.com/apps/sonarcloud

https://codeql.github.com/

see the repo for reference
DiamondLightSource/i18-bluesky#20

There is a DLS precedent for the use of codeql in the python-murfey repo

UPDATE: this had also been tested in the i18-bluesky repository

@stan-dot stan-dot added documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed dependencies Pull requests that update a dependency file labels Aug 7, 2024
@coretl
Copy link
Contributor

coretl commented Aug 19, 2024

Before working on a PR to put this into copier, please try it for a reasonable period in other repos, then report back if it gives any useful results. My experience with these tools is that they make more noise than useful alerts, so I'd like to see if they have improved.

@stan-dot
Copy link
Contributor Author

that's very reasonable.

arguably this should be an existing repo, not a new one.

@coretl , @callumforrester do you have a repo candidate? maybe blueapi ?

@DiamondJoseph
Copy link

Be nice to prove we have it configured right on i22-bluesky, since right now the security tab isn't really giving much information.

@callumforrester
Copy link
Contributor

Yep, happy to see this added to blueapi

@stan-dot
Copy link
Contributor Author

will revisit this once those repos are tested for 3 months - setting this for 19.11.2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants