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

Update CI piplines for environment #1217

Merged
merged 2 commits into from
Aug 15, 2017
Merged

Update CI piplines for environment #1217

merged 2 commits into from
Aug 15, 2017

Conversation

irfanhabib
Copy link
Contributor

No description provided.

Copy link
Contributor

@nwmac nwmac left a comment

Choose a reason for hiding this comment

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

LGTM - reviewed with Irfan

@nwmac nwmac merged commit 7f3c976 into master Aug 15, 2017
@nwmac nwmac deleted the change-e2e-registry branch August 15, 2017 09:12
KlapTrap added a commit that referenced this pull request Aug 17, 2017
* master:
  Fix style issue
  Persist app wall selection of cf/org/space in local storage (#1214)
  Add upgrade documentation for helm repository based installation (#1218)
  Fixed docker image name for all-in-one deployment (#1216)
  Use stable name for docker registry (#1220)
  Update CI piplines for environment (#1217)
  Fixes and changed select source radio button to select-input dropdown - Converted the select source radio button to a drop down (ready for more sources) - Split out more styles from deploy-step-source.scss - Fixed github userinput validation - Fixed display of cf ignore info for manually opened file/folders
  Deploy fixes, removed common session data, added delay to 'scanning' busy screen
  Started deploy step, a few minor bugs to work through WIP
  UX for source mostly works now. Still plenty of tidy up to do WIP
  Refactor/reduce deploy app service WIP - Move steps out into their own service - Move input sources out into seperate directives
  Split deploy destination + source step into two steps
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants