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

[documentation] Be a little bit verbose with aliases and setup #1524

Merged
merged 1 commit into from Jun 6, 2019
Merged

[documentation] Be a little bit verbose with aliases and setup #1524

merged 1 commit into from Jun 6, 2019

Conversation

cbastienbaron
Copy link
Contributor

change orders actions to avoid downtime and add some usefull informations

@cbastienbaron cbastienbaron changed the title Be a little bit verbose [documentation] Be a little bit verbose with aliases and setup May 28, 2019
@XWB
Copy link
Member

XWB commented May 29, 2019

@cbastienbaron Can you fix the merge conflict?

change orders actions to avoid downtime and add some usefull informations
@cbastienbaron
Copy link
Contributor Author

@XWB hoo :) done

@cbastienbaron
Copy link
Contributor Author

workflow tested in production like described in this PR, works like a charm ! 👍
Any reason to not merge ?

@XWB
Copy link
Member

XWB commented Jun 6, 2019

Thanks; no particular reason, just didn't find the time yet :)

@XWB XWB merged commit 73826f9 into FriendsOfSymfony:master Jun 6, 2019
@cbastienbaron
Copy link
Contributor Author

👍 Ty @XWB

@cbastienbaron cbastienbaron deleted the patch-2 branch June 6, 2019 14:50
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.

None yet

2 participants