Staging template cleanup. #938

Merged
merged 3 commits into from Feb 21, 2014

Conversation

Projects
None yet
2 participants
Contributor

richardkmichael commented Feb 21, 2014

No description provided.

Owner

leehambley commented Feb 21, 2014

@leehambley leehambley added a commit that referenced this pull request Feb 21, 2014

@leehambley leehambley Merge pull request #938 from richardkmichael/master
Staging template cleanup.
4c62998

@leehambley leehambley merged commit 4c62998 into capistrano:master Feb 21, 2014

1 check passed

default The Travis CI build passed
Details

@miry miry added a commit to miry/capistrano that referenced this pull request Apr 23, 2014

@juanibiapina @miry juanibiapina + miry 🐼 Added tests for after/before hooks
capistrano#1004

The changes of #938 affect the order the task executes. The "prerequisite" task seems to be running as
if it was defined in a after block, which makes sense, since it is now invoked by the task instead of being a prerequisite.

Since there were missed tests and @juanibiapina added simple tests in #1005, and were modified in this commit to cover
current business logic of the master.
e6f3526

@miry miry added a commit to miry/capistrano that referenced this pull request Apr 23, 2014

@juanibiapina @miry juanibiapina + miry 🐼 Added tests for after/before hooks
capistrano#1004

The changes of #938 affect the order the task executes. The "prerequisite" task seems to be running as
if it was defined in a after block, which makes sense, since it is now invoked by the task instead of being a prerequisite.

Since there were missed tests and @juanibiapina added simple tests in #1005, and were modified in this commit to cover
current business logic of the master.
9da7eb5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment