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.

Sign up for GitHub

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

Jump to bottom

Fix deployment pipeline #49

Merged
merged 1 commit into from
May 14, 2018
Merged

Fix deployment pipeline #49

merged 1 commit into from
May 14, 2018

Conversation

aknysh
Copy link
Member

@aknysh aknysh commented May 14, 2018

what

  • Fix pipeline to deploy only on tagged releases

why

Verifying all conditions in all clause are true...                                                                                                      
Evaluating execution condition executeForMasterBranch ( '0.2.2' == 'master' ): it evaluates to false                                                    
Evaluating execution condition executeForTag ( '0.2.2' != '' ): it evaluates to true                                                                    
Skipping execution.

@aknysh aknysh self-assigned this May 14, 2018
@aknysh aknysh requested a review from osterman May 14, 2018 18:32
@osterman osterman changed the title Fix pipeline Fix deployment pipeline May 14, 2018
@aknysh aknysh merged commit 65d6b2b into master May 14, 2018
@aknysh aknysh deleted the fix-pipeline branch May 14, 2018 18:36
goruha added a commit that referenced this pull request May 15, 2018
@goruha
Merge branch 'add-asciinema' of github.com:cloudposse/docs into add-a…
c79b355 
…sciinema

* 'add-asciinema' of github.com:cloudposse/docs:
  Switch to Creative Commons Attribution-NonCommercial-ShareAlike 4.0 International License (#53)
  Add Google Verification File (#51)
  Fix pipeline (#49)
  Only deploy on tagged releases to `master` branch (#48)
  Mobile responsive css tweaks (#41)
  Fix image copy/paste (#45)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@osterman osterman osterman approved these changes

Assignees

@aknysh aknysh

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aknysh @osterman