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

Add command task to execute Capistrano tasks #48

Merged
merged 7 commits into from
Jun 23, 2017
Merged

Conversation

MinasMazar
Copy link
Contributor

@MinasMazar MinasMazar commented Apr 11, 2017

In ref to #44 Pulsar can now execute Capistrano tasks.

@MinasMazar MinasMazar force-pushed the capistrano-tasks branch 2 times, most recently from c46d282 to e1f46b5 Compare April 11, 2017 15:44
@MinasMazar MinasMazar force-pushed the capistrano-tasks branch 2 times, most recently from 4ea4ddd to 18053aa Compare April 20, 2017 11:59
subject do
described_class.call(
cap_path: './cap-path', config_path: './config-path',
bundle_path: './bundle-path', environment: 'production'
bundle_path: './bundle-path', environment: 'production',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra comma

it { is_expected.to match(output) }
end


Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra blank line detected.

let(:environment) { 'staging' }
let(:output) { /Executed task deploy:check for your_app on staging!\n/ }


Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra blank line detected.

@MinasMazar MinasMazar force-pushed the capistrano-tasks branch 3 times, most recently from 08e5ba5 to dfd0147 Compare April 26, 2017 11:41
@mtylty mtylty merged commit d3e8141 into master Jun 23, 2017
@mtylty mtylty deleted the capistrano-tasks branch June 23, 2017 13:00
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