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

security: use forked install-composer to support channel installation #66

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

pierreboissinot
Copy link
Member

@pierreboissinot pierreboissinot commented Sep 19, 2024

J'ai fait une PR sur cbrunnkvist/ansistrano-symfony-deploy mais le projet n'est plus maintenu.

Nous avons deux solutions:

  • merge dans le fork lephare et utilisé ce fork dans ansible-deploy (donc version master et non install-composer)
  • attendre ? Le projet n'a pas bougé depuis 4 ans

Je suis pour changer les requirements de ansible-deploy pour utiliser notre fork.

Qu'en dites-vous ?

J'ai testé l'image docker les changements fonctionnent comme attendu.

@lucasmirloup
Copy link
Member

Je valide l'idée de coder sur notre fork (https://github.com/le-phare/ansistrano-symfony-deploy), et de l'utiliser sur ce repo

meta/main.yml Outdated Show resolved Hide resolved
Co-authored-by: Lucas Mirloup <[email protected]>
@pierreboissinot
Copy link
Member Author

@thislg @lucasmirloup unef ois que le-phare/ansistrano-symfony-deploy#1 est merged, je veux bien un coup de main pour savoir ce qu'il faut faire pour la prochaine release. depuis qu'on utilise "make release" je crois je n'en n'ai pas faite.

@pierreboissinot
Copy link
Member Author

@thislg @lucasmirloup PR merged sur le repo origin

cbrunnkvist/ansistrano-symfony-deploy#40 (comment)

Malgré cela on devrait basculer sur notre fork, nous aurons sans doute la mm situation à l'avenir et s'il faut appuyer sur "sync fork" 1 fois tous les 4 ans ça se gère =)

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.

3 participants