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

[#58] Upgrade TicketBooth from Rails 4.2 to Rails 6.1 #66

Closed
wants to merge 31 commits into from

Conversation

kigster
Copy link
Collaborator

@kigster kigster commented Mar 12, 2024

NOTE: some unfinished changes were accidentally merged into main. So this PR builds on top of those changes.

Gradual upgrade of Rails:

  • Rails 4.2 ➜ Rails 6.1 [In Progress]
  • Rails 6.1 ➜ Rails 7.1 [Not yet started]

Status

Not ready for reviews.

* Create rubyonrails.yml

* [WIP] Rails 6.0.2 & Ruby 3.2.3

* Fixing a bunch of failing specs

* Fix event_admin specs
@kigster kigster requested a review from a team as a code owner March 12, 2024 18:44
@kigster kigster self-assigned this Mar 16, 2024
@kigster kigster changed the title WIP: rails 6, ruby 3.2 [#58] Upgrade TicketBooth from Rails 4.2 to Rails 7.1 Mar 16, 2024
@kigster kigster changed the title [#58] Upgrade TicketBooth from Rails 4.2 to Rails 7.1 [#58] Upgrade TicketBooth from Rails 4.2 to Rails 6.1 Mar 17, 2024
@kigster kigster force-pushed the kig/rails-6-ruby-3.2 branch 2 times, most recently from 03daf16 to 233ffc3 Compare March 17, 2024 20:19
 * removing Sham
 * adding Faker gem
 * switching from custom made make! to build() using FactoryBot
 * fixing submissions link generator
@kigster kigster removed the request for review from a team March 17, 2024 20:43
@kigster kigster force-pushed the kig/rails-6-ruby-3.2 branch 2 times, most recently from ddd0cf4 to 5aa46c4 Compare March 26, 2024 01:20
@kigster kigster changed the base branch from main to kig/rails-7 April 9, 2024 16:46
@kigster kigster changed the base branch from kig/rails-7 to main April 9, 2024 16:47
@kigster kigster closed this Apr 9, 2024
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.

1 participant