Skip to content
This repository was archived by the owner on May 6, 2020. It is now read-only.

fix(api): fail when rolling back to v1#762

Merged
bacongobbler merged 3 commits intodeis:masterfrom
bacongobbler:fail-rollback-v1
Jul 12, 2016
Merged

fix(api): fail when rolling back to v1#762
bacongobbler merged 3 commits intodeis:masterfrom
bacongobbler:fail-rollback-v1

Conversation

@bacongobbler
Copy link
Copy Markdown
Member

Summary of Changes

v1 is an intial release created when deis create is called, but
does not contain any build information.

Issue(s) that this PR Closes

addresses deis/workflow#275 (comment)

Associated End To End Test PR(s)

N/A

Associated Documentation PR(s)

N/A

Pull Request Hygiene TODOs

Please make sure the below checklist is complete.

  • Your pull request is concise and to the point (make another PR for refactoring nearby code)
  • Your commits are squashed into logical units of work
  • Your commits follow the commit style guidelines

🌸 Thank you! 🌸

@mention-bot
Copy link
Copy Markdown

By analyzing the blame information on this pull request, we identified @mboersma, @Joshua-Anderson and @helgi to be potential reviewers

@kmala kmala added the LGTM1 label May 31, 2016
@bacongobbler bacongobbler force-pushed the fail-rollback-v1 branch 4 times, most recently from 609fc7b to ee5b338 Compare June 3, 2016 23:18
@helgi
Copy link
Copy Markdown
Contributor

helgi commented Jun 4, 2016

Unit tests are failing on this one

@bacongobbler
Copy link
Copy Markdown
Member Author

Yeah I'll have to get around for this one. It'd be nice to have for 2.0 but not a priority

@bacongobbler bacongobbler self-assigned this Jul 8, 2016
@bacongobbler bacongobbler added this to the v2.2 milestone Jul 8, 2016
@bacongobbler bacongobbler force-pushed the fail-rollback-v1 branch 2 times, most recently from 13d3660 to f253094 Compare July 8, 2016 19:02
v1 is an intial release created when `deis create` is called, but
does not contain any build information.
@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 11, 2016

Current coverage is 86.73%

Merging #762 into master will increase coverage by 0.01%

@@             master       #762   diff @@
==========================================
  Files            30         30          
  Lines          2749       2760    +11   
  Methods           0          0          
  Messages          0          0          
  Branches        454        457     +3   
==========================================
+ Hits           2384       2394    +10   
- Misses          239        240     +1   
  Partials        126        126          

Powered by Codecov. Last updated by 7647569...784cc42

@bacongobbler
Copy link
Copy Markdown
Member Author

the github UI is showing red, yet the travis build went green. Go figure. Merging!

@bacongobbler bacongobbler merged commit d4415c9 into deis:master Jul 12, 2016
@bacongobbler bacongobbler deleted the fail-rollback-v1 branch July 12, 2016 16:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants