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

Run brakeman in GitHub Actions #23713

Merged
merged 3 commits into from
Jul 11, 2023
Merged

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Feb 19, 2023

Try using GitHub and CodeQL for brakeman results

@nschonni nschonni marked this pull request as draft February 19, 2023 01:00
@nschonni

This comment was marked as outdated.

@nschonni nschonni force-pushed the brakeman-ci branch 4 times, most recently from 37c5ad8 to 11ba3d3 Compare February 19, 2023 02:05
@nschonni

This comment was marked as outdated.

@Gargron
Copy link
Member

Gargron commented Feb 19, 2023

FYI I already removed CodeClimate from the repository so you can remove its configuration file.

@nschonni
Copy link
Contributor Author

FYI I already removed CodeClimate from the repository so you can remove its configuration file.

Thanks, I had dropped that one, but it created a bunch of issues, so I put it back in.
I've run into an issue with the sarif uploads to CodeQL not recogonizing the suppressions in breakman.ignore, so I may go back to the simpler approach

@nschonni nschonni marked this pull request as ready for review February 19, 2023 07:30
@nschonni
Copy link
Contributor Author

@Gargron I deleted the config file again, but it looks like the application is still installed in the GitHub App Settings, since it's showing the failed hook results.

@nschonni nschonni force-pushed the brakeman-ci branch 3 times, most recently from ecaa890 to 17aba12 Compare March 7, 2023 20:03
@nschonni nschonni marked this pull request as draft March 7, 2023 20:11
@nschonni nschonni marked this pull request as ready for review March 7, 2023 20:18
@nschonni
Copy link
Contributor Author

nschonni commented Mar 7, 2023

Because of the unrelated CI failure on #24019, I figured it was a better idea to split out the bundler-audit to it's own job, that only runs when the Gemfile changes + on a regular schedule

@nschonni
Copy link
Contributor Author

@Gargron is it easier if I restore the codeclimate file and just disable the last brakeman config?

@ClearlyClaire
Copy link
Contributor

oops, sorry I missed that, and I'm actually surprised we've stopped running brakeman… when did that happen?

Is there a way we can have brakeman's results more prominent and readable, instead of having them buried in the Ruby Linting action logs?

@renchap
Copy link
Sponsor Member

renchap commented Jun 1, 2023

Maybe with something like https://github.com/reviewdog/action-brakeman ?

@github-actions
Copy link
Contributor

github-actions bot commented Jul 8, 2023

This pull request has merge conflicts that must be resolved before it can be merged.

Allows it to run on a cron schedule to see about new issues outside of a PR run
@github-actions
Copy link
Contributor

github-actions bot commented Jul 8, 2023

This pull request has resolved merge conflicts and is ready for review.

@nschonni
Copy link
Contributor Author

nschonni commented Jul 8, 2023

oops, sorry I missed that, and I'm actually surprised we've stopped running brakeman… when did that happen?

When codeclimate was removed

@ClearlyClaire ClearlyClaire merged commit e110325 into mastodon:main Jul 11, 2023
29 checks passed
@nschonni nschonni deleted the brakeman-ci branch July 11, 2023 14:50
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.

None yet

5 participants