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

Added support for .tool-versions file in Gemfile for Ruby projects #9070

Merged
merged 3 commits into from May 24, 2024

Conversation

kpumuk
Copy link
Contributor

@kpumuk kpumuk commented Feb 16, 2024

This MR is related to #1033 in a way that it adds support for .tool-versions files for Ruby projects.

Starting with Bunder 2.4.20, it supports specifying Ruby version via .tool-versions (see rubygems/rubygems#6898 and Specifying a Ruby Version). Unfortunately, Dependabot does not download this file and does not parse Ruby version from it.

This MR enables Ruby projects to specify Ruby version in a single file.

@kpumuk kpumuk requested a review from a team as a code owner February 16, 2024 19:09
@github-actions github-actions bot added the L: ruby:bundler RubyGems via bundler label Feb 16, 2024
@@ -0,0 +1 @@
!.tool-versions
Copy link
Contributor Author

Choose a reason for hiding this comment

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

.tool-versions is ignored at the root of the project, and so we need to negate it here to add this file to the fixture project.

@kpumuk kpumuk force-pushed the bundler-tool-versions branch 3 times, most recently from 34af3a4 to bc29131 Compare March 2, 2024 13:37
@kpumuk kpumuk force-pushed the bundler-tool-versions branch 2 times, most recently from 9d122b6 to 3eb8fbd Compare March 14, 2024 04:02
@kpumuk kpumuk force-pushed the bundler-tool-versions branch 2 times, most recently from dd8af5c to 9a3cfeb Compare March 23, 2024 02:24
@janklimo
Copy link

Looking forward to this! Thank you @kpumuk

@kpumuk
Copy link
Contributor Author

kpumuk commented Mar 31, 2024

Looking forward to this! Thank you @kpumuk

You and me both :-) I hope this will eventually get attention from maintainers.

@kpumuk kpumuk force-pushed the bundler-tool-versions branch 2 times, most recently from 2c4b4b1 to 06e3e26 Compare April 18, 2024 00:53
@janklimo
Copy link

@deivid-rodriguez seeing you were involved in adding support for this in rubygems, can you help move this forward by reviewing or assigning the right person? Dependabot is still broken for us and many others:

Screenshot 2024-05-15 at 10 58 42

@deivid-rodriguez
Copy link
Contributor

deivid-rodriguez commented May 16, 2024

Hei, I no longer maintain Dependabot.

This seems tested and ready to be reviewed though, so since it's been hanging for a couple months, I think it's fine to politely ping some other maintainer and ask for a review.

The Dependabot team is usually super keen on taking community contributions, they are just very busy 😅.

@janklimo
Copy link

@robaiken @landongrindheim you guys have done a lot of great work on the Ruby side of things of Dependabot. Would you have a moment to review/merge?

@kpumuk
Copy link
Contributor Author

kpumuk commented May 18, 2024

@jurre rebased main and reworded tests to address linter errors in https://github.com/dependabot/dependabot-core/actions/runs/9117390435/job/25068129813

@sachin-sandhu sachin-sandhu requested a review from jurre May 19, 2024 23:28
@sachin-sandhu
Copy link
Contributor

Hi @jurre , Request you to have a quick review and approve as @kpumuk has added new commits after last approval. Thank you.

@jurre
Copy link
Member

jurre commented May 21, 2024

Thanks everyone!

@sachin-sandhu sachin-sandhu merged commit 88b1efa into dependabot:main May 24, 2024
41 checks passed
@bewuethr
Copy link

bewuethr commented May 24, 2024

Omg it's been merged! Thank you all 🥳

Edit: works like a charm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: ruby:bundler RubyGems via bundler
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants