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

Fix broken CI #1353

Merged
merged 2 commits into from
Jun 15, 2023
Merged

Fix broken CI #1353

merged 2 commits into from
Jun 15, 2023

Conversation

tk0miya
Copy link
Contributor

@tk0miya tk0miya commented Jun 13, 2023

Our CI has been broken because of changes in Ruby 3.3.

This fixes them and gets green again.

stringio-3.0.7 was released at 2023/6/2 to support Ruby 3.3.
https://github.com/ruby/stringio/blob/master/NEWS.md#307---2023-06-02

This upgrade it from 3.0.6 to 3.0.7 to resolve errors for
master-nightly-* tests.
@tk0miya
Copy link
Contributor Author

tk0miya commented Jun 14, 2023

Oops. The new racc and rubocop package were released last week. So it's better to upgrade rubocop instead of pinning racc.
rubocop/rubocop#11942

Since Ruby 3.3, racc has been promoted as bundled gems (see ruby#7877).
Therefore, this updates the rubocop and racc to make Ruby 3.3 CI
succeeded.

refs: rubocop/rubocop#11942
@hsbt
Copy link
Member

hsbt commented Jun 15, 2023

👍

@hsbt hsbt added this pull request to the merge queue Jun 15, 2023
@koic
Copy link
Contributor

koic commented Jun 15, 2023

Exactly. Racc will be installed as a dependency via Parser gem if RuboCop 1.52.1+ is used.

Merged via the queue into ruby:master with commit 647a24f Jun 15, 2023
31 checks passed
@tk0miya tk0miya deleted the fix_ci branch June 15, 2023 02:04
@soutaro soutaro added this to the RBS 3.2 milestone Jul 18, 2023
soutaro pushed a commit that referenced this pull request Jul 18, 2023
Fix broken CI

# Conflicts:
#	Gemfile.lock
@soutaro soutaro mentioned this pull request Jul 18, 2023
@soutaro soutaro added the Released PRs already included in the released version label Aug 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Released PRs already included in the released version
Development

Successfully merging this pull request may close these issues.

None yet

4 participants