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

feat(whitelist): Add support for IP whitelist#1010

Merged
kmala merged 1 commit intodeis:masterfrom
kmala:maint
Aug 26, 2016
Merged

feat(whitelist): Add support for IP whitelist#1010
kmala merged 1 commit intodeis:masterfrom
kmala:maint

Conversation

@kmala
Copy link
Copy Markdown
Contributor

@kmala kmala commented Aug 24, 2016

ref #240

@deis-bot
Copy link
Copy Markdown

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

@kmala kmala self-assigned this Aug 24, 2016
@kmala kmala added this to the v2.5 milestone Aug 24, 2016
@codecov-io
Copy link
Copy Markdown

codecov-io commented Aug 24, 2016

Current coverage is 86.49% (diff: 95.83%)

Merging #1010 into master will increase coverage by 0.35%

@@             master      #1010   diff @@
==========================================
  Files            28         28          
  Lines          3175       3244    +69   
  Methods           0          0          
  Messages          0          0          
  Branches        540        552    +12   
==========================================
+ Hits           2735       2806    +71   
+ Misses          292        289     -3   
- Partials        148        149     +1   

Powered by Codecov. Last update 757a8ae...cc0a4d5

Comment thread rootfs/api/models/app.py
'app_type': scale_type,
'build_type': release.build.type,
'healthcheck': healthcheck,
'service_annotations': {'maintenance': app_settings.maintenance},
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why the removal here?

Copy link
Copy Markdown
Contributor Author

@kmala kmala Aug 25, 2016

Choose a reason for hiding this comment

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

its unused...thought i can cleanup a bit.

@helgi helgi added the LGTM1 label Aug 25, 2016
@kmala kmala merged commit 9b8ede9 into deis:master Aug 26, 2016
@kmala kmala deleted the maint branch August 26, 2016 19:08
@helgi helgi mentioned this pull request Aug 26, 2016
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