-
Notifications
You must be signed in to change notification settings - Fork 5
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
Conversation
En attente du merge de rspec/rspec-rails#2546 Après ça c'est OK à priori. |
Pour la ref, l'erreur:
Les 4 tests qui failent:
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 |
Je vais quand même faire tourner |
Bump en 7.0.1 |
La PR rspec/rspec-rails#2546 a été merge, en attente de la release de rspec-rails 6.x du coup. |
L'issue à suivre: rspec/rspec-rails#2560 imo cette semaine ça devrait être OK |
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 |
Clairement spring on peut flip, c'est plus en défaut dans rails 7 car les machines sont assez performantes maintenant. |
Je pense que #355 solve le dernier problème, on va pouvoir merge |
Oopah 🤷 |
Y'a une RC rspec/rspec-rails#2560 (comment) je rebase voir |
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 |
No description provided.