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

Fix Gitlab deploy step #3469

Merged
merged 3 commits into from
Apr 8, 2019
Merged

Fix Gitlab deploy step #3469

merged 3 commits into from
Apr 8, 2019

Conversation

richard-cox
Copy link
Contributor

@cfdreddbot
Copy link

✅ Hey richard-cox! The commit authors and yourself have already signed the CLA.

@codecov
Copy link

codecov bot commented Apr 4, 2019

Codecov Report

Merging #3469 into v2-master will increase coverage by <.01%.
The diff coverage is 72.46%.

@@              Coverage Diff              @@
##           v2-master    #3469      +/-   ##
=============================================
+ Coverage      51.61%   51.61%   +<.01%     
=============================================
  Files            698      701       +3     
  Lines          19465    19506      +41     
  Branches        3447     3455       +8     
=============================================
+ Hits           10046    10068      +22     
- Misses          9419     9438      +19

@richard-cox richard-cox changed the base branch from v2-master to 2.0.0-base April 5, 2019 13:43
@richard-cox richard-cox changed the base branch from 2.0.0-base to v2-master April 5, 2019 13:43
@KlapTrap KlapTrap merged commit ead1146 into v2-master Apr 8, 2019
KlapTrap added a commit that referenced this pull request Apr 18, 2019
* v2-master: (41 commits)
  Tweaks following project update (#3476)
  Update status_updates.md
  Fix unit tests
  don't show the favourite list line seperator on the last set of entities
  Only show the filter toggle if there's something to filter on
  Add angular sanitizer to marked renderer
  Bump marked dependency to fix snyk error
  Address review comments
  Fix Gitlab deploy step (#3469)
  Update status_updates.md
  Fix after merge
  fix e2e test
  Fix unit tests (#3466)
  Fix unit tests - not sure how these are broken on master, all PRs have been passing - fix is to ensure endpoint & home page component tests use pre-populated store - also improved typing
  Removed fdescribe
  Fix e2e tests
  Tidy up types file
  Add specific help text to deploy step
  Add specific deploy types in base tile selector
  Ensure you can't press cancel on endpoint connect dialog while it's busy
  ...
@nwmac nwmac deleted the fix-deploy-step-failures branch April 10, 2020 11:20
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.

3 participants