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 Gemfile.lock file #165

Merged
merged 1 commit into from
Apr 9, 2019
Merged

Add Gemfile.lock file #165

merged 1 commit into from
Apr 9, 2019

Conversation

lbalmaceda
Copy link
Contributor

@lbalmaceda lbalmaceda commented Mar 21, 2019

Changes

This PR explicitly adds the Gemfile.lock file required for Snyk to report vulnerabilities. Ideally this should run as part of a CI build. I'll migrate this soon to CircleCI.

References

Snyk documentation for Ruby

Testing

You can run the check locally using the Snyk CLI.

snyk test

Checklist

@joshcanhelp
Copy link
Contributor

@lbalmaceda - Another set of failing tests here. I seem to recall a similar issue when I tried to add the lock file before. I will investigate a bit on my end.

@lbalmaceda
Copy link
Contributor Author

@joshcanhelp thanks! Ping me if you need help.

Copy link
Contributor

@joshcanhelp joshcanhelp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

More changes coming to fix CI tests failing

@joshcanhelp joshcanhelp merged commit faa26b3 into master Apr 9, 2019
@joshcanhelp joshcanhelp added this to the v4-Next milestone Apr 9, 2019
@joshcanhelp joshcanhelp deleted the lock-patch branch July 30, 2019 22:03
@joshcanhelp joshcanhelp changed the title add Gemfile.lock file Add Gemfile.lock file Jul 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants