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

Relax matching pattern for rake version #7123

Merged
merged 1 commit into from Oct 30, 2023
Merged

Relax matching pattern for rake version #7123

merged 1 commit into from Oct 30, 2023

Conversation

hsbt
Copy link
Member

@hsbt hsbt commented Oct 30, 2023

What was the end-user or developer problem that led to this PR?

I released Rake 13.1.0 in last weekend. After that, some of bundler example is failing because they used 13.0 pattern for checking version number.

What is your fix for the problem, implemented in this PR?

I replaced it with \d regex.

Make sure the following tasks are checked

@hsbt hsbt enabled auto-merge October 30, 2023 03:33
@hsbt hsbt merged commit 5acec1a into master Oct 30, 2023
92 checks passed
@hsbt hsbt deleted the relax-rake branch October 30, 2023 04:46
@simi
Copy link
Member

simi commented Oct 30, 2023

@deivid-rodriguez uhh, would be good to change the test to not rely on rack latest version somehow to not break those tests again with rake 14 🙏

@deivid-rodriguez
Copy link
Member

Sure, or even better make those tests don't rely on the release schedule of real gems. Any improvements there super welcome!

deivid-rodriguez pushed a commit that referenced this pull request Nov 8, 2023
Relax matching pattern for rake version

(cherry picked from commit 5acec1a)
deivid-rodriguez pushed a commit that referenced this pull request Nov 8, 2023
Relax matching pattern for rake version

(cherry picked from commit 5acec1a)
deivid-rodriguez pushed a commit that referenced this pull request Nov 8, 2023
Relax matching pattern for rake version

(cherry picked from commit 5acec1a)
deivid-rodriguez pushed a commit that referenced this pull request Nov 8, 2023
Relax matching pattern for rake version

(cherry picked from commit 5acec1a)
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.

None yet

3 participants