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

Support loading slim/smart #1582

Merged
merged 1 commit into from Apr 22, 2021
Merged

Support loading slim/smart #1582

merged 1 commit into from Apr 22, 2021

Conversation

presidentbeef
Copy link
Owner

in the very specific case where slim/smart is mentioned in the Gemfile.

If others need this functionality, it can be expanded. It's just awkward to find where apps require things with the way Brakeman works, if you need to know early in the analysis. So, this is a hack.

Fixes #1570

in a very specific case.

Fixes #1570
@presidentbeef presidentbeef merged commit 05e2372 into main Apr 22, 2021
@presidentbeef presidentbeef deleted the slim_smart branch April 22, 2021 20:16
Repository owner locked and limited conversation to collaborators Jan 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Slim parsing error "unknown line indicator" when using "slim/smart" plugin
1 participant