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

chore(deps): upgrade to Rails 7.1 #7517

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

adi-herwana-nus
Copy link

No description provided.

Gemfile Outdated
@@ -183,7 +183,7 @@ gem 'cancancan'
# Some helpers for structuring CSS/JavaScript
# Official version https://github.com/winston/rails_utils/pull/30 is no longer maintained.
# We also want stricter sanitization.
gem 'rails_utils', git: 'https://github.com/Coursemology/rails_utils.git'
# gem 'rails_utils', git: 'https://github.com/Coursemology/rails_utils.git'
Copy link
Contributor

Choose a reason for hiding this comment

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

if this gem is no longer required, better to just delete this line (and also the comments above that are explaining why this gem is needed) instead of just commenting it out, for the hygiene purpose of this file

@adi-herwana-nus adi-herwana-nus changed the title chore(deps): remove unused rails_utils dependency chore(deps): upgrade to Rails 7.1 Sep 10, 2024
@cysjonathan cysjonathan mentioned this pull request Sep 10, 2024
7 tasks
@adi-herwana-nus adi-herwana-nus force-pushed the adi/test-remove-rails-utils branch 2 times, most recently from c22224d to 7dcafbc Compare September 11, 2024 06:57
adi-herwana-nus added 2 commits September 16, 2024 14:08
- removed deprecated fixture_path config keys
- updated certain enum fields to be attributes
- changed @connection to @raw_connection in database extension
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