Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Replace Capifony with Capistrano/symfony#5533
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Since Capifony only works with Capistrano 2 (https://github.com/everzet/capifony/blob/master/README.md) and Capistrano 3 is the current version, its probably better to advice using the newer alternative.
cookbook/deployment/tools.rst Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
symphony while contributing to the Symfony documentation ? not good...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
sorry :)
cookbook/deployment/tools.rst Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
you still have symphony here
wouterj commentedJul 21, 2015
👍 (there are 2 minor spacing issues: one space too much before |
xabbuh commentedJul 21, 2015
Do we really want to dismiss Capifony? Why don't we promote both? |
wouterj commentedJul 21, 2015
@xabbuhhttp://williamdurand.fr/2015/04/11/on-capifony-and-its-future/ suggests that Capifony is there only for backwards compability and new projects should use either Capistrano v3 or other tools (like Docker). As this message is written by the maintainers of Capifony, I think it's better to no longer mention it (and btw, there still is a mention of Capifony). |
xabbuh commentedJul 21, 2015
indeed, I should have thought about it 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
where is the Capifony link defined?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
on line 192/193 (it already existed, so the diff doesn't show it)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
thank you@wouterj
wouterj commentedJul 23, 2015
Cool! Thanks for keeping the docs in sync with changes in Symfony tooling@mojzis! |
This PR was submitted for the 2.7 branch but it was merged into the 2.3 branch instead (closes#5533).Discussion----------Replace Capifony with Capistrano/symfonySince Capifony only works with Capistrano 2 (https://github.com/everzet/capifony/blob/master/README.md) and Capistrano 3 is the current version, its probably better to advice using the newer alternative.Commits-------8f3ec0c Replace Capifony with Capistrano/symfony
Since Capifony only works with Capistrano 2 (https://github.com/everzet/capifony/blob/master/README.md) and Capistrano 3 is the current version, its probably better to advice using the newer alternative.