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

FI-2160 Updated dependencies #51

Closed
wants to merge 3 commits into from
Closed

Conversation

360dgries
Copy link
Contributor

@360dgries 360dgries commented Nov 17, 2023

Completion of FI_2160 for this Repo.

.simplecov has did not need to be commented out for this repo -- would be interested in thoughts on why that is.

Have also brought rubocop up to a version that can handle the newer versions of ruby. Had to add some steps to the job process as well, as it was having issues with the gemfile being frozen.

@360dgries 360dgries marked this pull request as draft November 17, 2023 18:47
@360dgries
Copy link
Contributor Author

Would like another set of eyes on why codeclimate is failing. Seems like it's mad that we left TODO comments in, but I think those comments are generated by rubocop to describe the keywords it's looking for?

@360dgries 360dgries marked this pull request as ready for review November 17, 2023 20:23
@360dgries 360dgries changed the title Updated dependencies FI-2160 Updated dependencies Nov 27, 2023
@Jammjammjamm
Copy link
Contributor

Replaced by #52

@Jammjammjamm Jammjammjamm deleted the FI-2160-Update-Dependencies branch December 5, 2023 15:36
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.

2 participants