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

fix(models): add "_added" log function, lowercase class name#1017

Merged
bacongobbler merged 1 commit intodeis:masterfrom
bacongobbler:lowercase-class-name
Aug 26, 2016
Merged

fix(models): add "_added" log function, lowercase class name#1017
bacongobbler merged 1 commit intodeis:masterfrom
bacongobbler:lowercase-class-name

Conversation

@bacongobbler
Copy link
Copy Markdown
Member

This change caused e2e to fail because it expects the class name to be lower case.

@bacongobbler bacongobbler added this to the v2.5 milestone Aug 26, 2016
@bacongobbler bacongobbler self-assigned this Aug 26, 2016
@deis-bot
Copy link
Copy Markdown

@helgi, @mboersma and @kmala are potential reviewers of this pull request based on my analysis of git blame information. Thanks @bacongobbler!

@codecov-io
Copy link
Copy Markdown

codecov-io commented Aug 26, 2016

Current coverage is 86.68% (diff: 100%)

Merging #1017 into master will not change coverage

@@             master      #1017   diff @@
==========================================
  Files            29         29          
  Lines          3282       3282          
  Methods           0          0          
  Messages          0          0          
  Branches        555        555          
==========================================
  Hits           2845       2845          
  Misses          288        288          
  Partials        149        149          

Powered by Codecov. Last update 50811a2...c300883

@helgi
Copy link
Copy Markdown
Contributor

helgi commented Aug 26, 2016

This change caused e2e to fail because it expects the class name to be lower case.
@bacongobbler bacongobbler changed the title fix(models): lowercase class name when logging fix(models): add "_added" log function, lowercase class name Aug 26, 2016
@bacongobbler bacongobbler merged commit 899e008 into deis:master Aug 26, 2016
@bacongobbler bacongobbler deleted the lowercase-class-name branch August 26, 2016 21:57
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.

5 participants