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

Commit *.lock files #276

Merged
merged 2 commits into from
Jul 22, 2023
Merged

Commit *.lock files #276

merged 2 commits into from
Jul 22, 2023

Conversation

wildmaples
Copy link
Collaborator

@wildmaples wildmaples commented Jul 14, 2023

Open source projects these days should commit their lockfiles. Not committing lockfiles is a fairly outdated policy/concept. With the advent of better CI and matrix builds, this Gemfile.lock policy is no longer the case. You’ll see a lot of popular gem projects now check in their lockfile. This is because it makes setting up development easier and more predictable. In 2010, CI practices were not as robust as they are today so using the “randomness” of no lockfile helped suss out edge cases.

@wildmaples wildmaples requested a review from ngan July 14, 2023 18:35
@wildmaples wildmaples force-pushed the commit-lock-files branch 2 times, most recently from 7cd44f1 to ecf21e0 Compare July 14, 2023 18:38
@wildmaples
Copy link
Collaborator Author

wildmaples commented Jul 14, 2023

Rails 6.0 is not compatible with Ruby 3.0 and above. Both Rails 6.0 and Ruby 2.7 has reached EOL. I propose we stop supporting version Rails 6.0.

The question of why/how the CI check was passing before will be another conversation 😬

Copy link
Collaborator

@ngan ngan left a comment

Choose a reason for hiding this comment

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

👍

@wildmaples wildmaples merged commit 22d2e37 into main Jul 22, 2023
13 checks passed
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

2 participants