-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Replace Capifony with Capistrano/symfony #5533
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.
`Capistrano`_, tailored specifically to Symfony projects. | ||
`Capistrano`_/`symphony`_ | ||
`Capistrano`_ is a remote server automation and deployment tool written in Ruby. | ||
`symphony`_ is a plugin to ease symphony related tasks, inspired by `Capifony`_ |
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 :)
`Capistrano`_, tailored specifically to Symfony projects. | ||
`Capistrano`_ with `symfony plugin`_ | ||
`Capistrano`_ is a remote server automation and deployment tool written in Ruby. | ||
`symfony plugin`_ is a plugin to ease symphony related tasks, inspired by `Capifony`_ |
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
👍 (there are 2 minor spacing issues: one space too much before |
Do we really want to dismiss Capifony? Why don't we promote both? |
@xabbuh http://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). |
indeed, I should have thought about it 👍 |
`Capistrano`_, tailored specifically to Symfony projects. | ||
`Capistrano`_ with `Symfony plugin`_ | ||
`Capistrano`_ is a remote server automation and deployment tool written in Ruby. | ||
`Symfony plugin`_ is a plugin to ease Symfony related tasks, inspired by `Capifony`_ |
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
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/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. 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.