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 CI Failures #60

Merged
merged 1 commit into from
Jan 17, 2025
Merged

Fix CI Failures #60

merged 1 commit into from
Jan 17, 2025

Conversation

mtakeda15
Copy link

What and Why

1: Can not build sqlite3
→ ubuntu 24.04.1 does not include libsqlite3-dev.
→ add sudo apt-get install libsqlite3-dev

2: rails db:setup fails
concurrent-ruby 1.3.5 has breaking changes. When use it with Rails < 7.1, rails command will fail.
→ use concurrent-ruby 1.3.4

@mtakeda15 mtakeda15 marked this pull request as ready for review January 17, 2025 08:08
@mtakeda15 mtakeda15 self-assigned this Jan 17, 2025
@mtakeda15 mtakeda15 requested a review from valerauko January 17, 2025 08:09
@@ -112,9 +112,11 @@ jobs:
BUNDLE_GEMFILE: ${{ github.workspace }}/.github/gemfiles/rails-${{ matrix.rails }}.Gemfile

steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4

Choose a reason for hiding this comment

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

ついで?

Comment on lines 5 to +6
gem "rails", "7.0.4.3"
gem 'concurrent-ruby', '1.3.4'

Choose a reason for hiding this comment

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

別のPRでもかまわんけど、対応するrailsバージョンは更新しましょう

## test against last two releases from supported rails versions
## when updating, be sure to add the matching version Gemfile to
## .github/gemfiles

https://endoflife.date/rails

不要になる

  • 5系
  • 6系

追加

※ actionファイルでは「サポートされてる系統(マイナー)の最新2つ」と書いてあるのでそれに合わせましょう

  • 7.1
  • 7.2
  • 8.0

これなら今回の対応は7.0系しか必要ない

Copy link
Author

Choose a reason for hiding this comment

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

確かに。別PRで5,6は消してしまいますね

with:
fetch-depth: 0
- name: install libsqlite3-dev
run: sudo apt-get install libsqlite3-dev

Choose a reason for hiding this comment

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

🚀

@mtakeda15 mtakeda15 merged commit 7ee668c into master Jan 17, 2025
30 checks passed
@mtakeda15 mtakeda15 deleted the feature/fix-ci branch January 17, 2025 16:01
This was referenced Jan 17, 2025
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