Skip to content

[Omniauth] Changement de stratégie d'autoload sur la gem #5036

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

Merged
merged 3 commits into from
Apr 28, 2025

Conversation

Michaelvilleneuve
Copy link
Collaborator

L'autoload des fichiers de la gem ne fonctionne pas une fois émise sur rubygems. Je change légèrement la stratégie, j'ai eu du succès avec celle-ci dans le passé.

@victormours
Copy link
Contributor

Je suis pas sûr de comprendre la description : est-ce que tu es sûr que ça marche avec ce code ou est-ce que c'est une tentative ? Si c'est le deuxième cas il vaut mieux modifier les fichiers en local (avec un bundle show omniauth-rdv-service-public) pour valider ça.

@Michaelvilleneuve
Copy link
Collaborator Author

Je suis pas sûr de comprendre la description : est-ce que tu es sûr que ça marche avec ce code ou est-ce que c'est une tentative ? Si c'est le deuxième cas il vaut mieux modifier les fichiers en local (avec un bundle show omniauth-rdv-service-public) pour valider ça.

@victormours En fait la Gem qui est actuellement sur Rubygems est vraiment cassée, les fichiers n'apparaissent même pas.

image

C'est pour ça que je suggérais cette modification en attendant d'avoir accès et de pouvoir publier une version pour faire un essai.
Pourras-tu m'ajouter comme collaborateur please 🙏 MichaelBeta

@victormours
Copy link
Contributor

Ah ok je comprends mieux. C'est donc surtout l'ajout de gem.files qui fait qu'on ajoute les fichiers correctement. Je me rends compte qu'on a le même problème sur la gem de l'anonymizer, je vais faire un autre fix pour ça.
Normalement tu as reçu une invitation à être maintainer sur la gem (ruby gems me dit qu'elle est "pending" pour MichaelBeta).

Copy link
Contributor

@victormours victormours left a comment

Choose a reason for hiding this comment

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

👍 merci pour le correctif !

@adipasquale
Copy link
Contributor

@Michaelvilleneuve est-ce que cette PR est toujours d’actualité ?

@Michaelvilleneuve
Copy link
Collaborator Author

@adipasquale Oui elle est toujours d'actualité, de fait de notre côté à cause de ce problème nous n'utilisons pas la Gem publiée sur Rubygems mais git: "https://github.com/betagouv/rdv-service-public.git", branch: "production", glob: "lib/omniauth-rdv-service-public/omniauth-rdv-service-public.gemspec"

@adipasquale adipasquale requested a review from victormours April 16, 2025 09:23
@adipasquale adipasquale enabled auto-merge (squash) April 28, 2025 12:56
@adipasquale adipasquale merged commit 52e9c18 into production Apr 28, 2025
25 of 26 checks passed
@adipasquale adipasquale deleted the fix/autoloading-omniauth-gem branch April 28, 2025 13:06
@github-project-automation github-project-automation bot moved this from 🚀 To deploy to ✅ Done in RDV Service Public Apr 28, 2025
adipasquale added a commit that referenced this pull request May 12, 2025
* changement de stratégie d'autoload

* minor version change

---------

Co-authored-by: Adrien Di Pasquale <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants