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

Bump to rails 7 (7.0.1) #266

Merged
merged 7 commits into from Apr 4, 2022
Merged

Bump to rails 7 (7.0.1) #266

merged 7 commits into from Apr 4, 2022

Conversation

skelz0r
Copy link
Member

@skelz0r skelz0r commented Dec 29, 2021

No description provided.

@skelz0r
Copy link
Member Author

skelz0r commented Dec 29, 2021

En attente du merge de rspec/rspec-rails#2546

Après ça c'est OK à priori.

@skelz0r
Copy link
Member Author

skelz0r commented Jan 6, 2022

Pour la ref, l'erreur:

     NameError:
       uninitialized constant ActionMailer::DeliveryJob
       Did you mean?  ActionMailer::MailDeliveryJob

Les 4 tests qui failent:

rspec './spec/features/create_magic_link_spec.rb[1:3:1:2]' # create a magic link when the current user is an admin behaves like it_creates_a_magic_link sends the email magic link
rspec './spec/features/create_magic_link_spec.rb[1:1:1:1:2]' # create a magic link when the current user is the token owner when the email address is valid behaves like it_creates_a_magic_link sends the email magic link
rspec './spec/features/admin/user_profile_spec.rb[1:2:6:1:1:2]' # admin user profile user tokens list magic token creation when the email address is valid behaves like it_creates_a_magic_link sends the email magic link
rspec ./spec/organizers/token/deliver_magic_link_to_email_spec.rb:20 # Token::DeliverMagicLinkToEmail with a valid email address queues the magic link email

Un exemple de code qui fail:

       expect { subject }
         .to have_enqueued_mail(JwtAPIEntrepriseMailer, :magic_link)
         .with(args: [email, token])

Si on veut vraiment bump en rails 7 ça peut se migrer izzy imo

@skelz0r
Copy link
Member Author

skelz0r commented Jan 6, 2022

Je vais quand même faire tourner rails app:update au cas où ça sera plus simple pour les futures bump.

@skelz0r skelz0r changed the title Bump to rails 7 Bump to rails 7.0.1 Jan 10, 2022
@skelz0r skelz0r changed the title Bump to rails 7.0.1 Bump to rails 7 (7.0.1) Jan 10, 2022
@skelz0r
Copy link
Member Author

skelz0r commented Jan 10, 2022

Bump en 7.0.1

@skelz0r
Copy link
Member Author

skelz0r commented Jan 11, 2022

La PR rspec/rspec-rails#2546 a été merge, en attente de la release de rspec-rails 6.x du coup.

@skelz0r
Copy link
Member Author

skelz0r commented Jan 31, 2022

L'issue à suivre: rspec/rspec-rails#2560

imo cette semaine ça devrait être OK

@Samuelfaure
Copy link
Contributor

Autant je suis pour virer 'spring-watcher-listen' (qui bloque spring en version 2 alors que la LTS est 4)

Autant, veut-on enlever spring complètement? Ça speed bien les tests il me semble

@skelz0r
Copy link
Member Author

skelz0r commented Feb 15, 2022

Clairement spring on peut flip, c'est plus en défaut dans rails 7 car les machines sont assez performantes maintenant.

@skelz0r
Copy link
Member Author

skelz0r commented Mar 14, 2022

Je pense que #355 solve le dernier problème, on va pouvoir merge

@skelz0r
Copy link
Member Author

skelz0r commented Mar 14, 2022

Oopah 🤷

@skelz0r
Copy link
Member Author

skelz0r commented Apr 4, 2022

Y'a une RC rspec/rspec-rails#2560 (comment) je rebase voir

@skelz0r
Copy link
Member Author

skelz0r commented Apr 4, 2022

C'est green, il a fallu que je fasse un semblant de bump global de dépendances (rien de bien méchant c'est quasiment que du dev/test et patch de gems global).

Si ça passe ici ça marge

@skelz0r skelz0r merged commit afdaeba into develop Apr 4, 2022
@skelz0r skelz0r deleted the bump/rails branch April 4, 2022 09:05
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.

None yet

2 participants