Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

Fix #1706: Add support for configuration of prometheus.io/path annota… #1707

Merged
merged 1 commit into from
Sep 19, 2019

Conversation

erikgb
Copy link
Contributor

@erikgb erikgb commented Sep 12, 2019

…tion

@rohanKanojia
Copy link
Member

@erikgb : Thanks, could you please add a line to CHANGELOG regarding this change?

@rohanKanojia rohanKanojia added the pr/changelog-entry-please Please add a changelog entry for this PR label Sep 12, 2019
@codecov
Copy link

codecov bot commented Sep 12, 2019

Codecov Report

Merging #1707 into master will increase coverage by 0.13%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1707      +/-   ##
============================================
+ Coverage     33.84%   33.98%   +0.13%     
  Complexity     1105     1105              
============================================
  Files           189      189              
  Lines         10690    10712      +22     
  Branches       1741     1745       +4     
============================================
+ Hits           3618     3640      +22     
  Misses         6650     6650              
  Partials        422      422

@codecov
Copy link

codecov bot commented Sep 12, 2019

Codecov Report

Merging #1707 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1707      +/-   ##
============================================
+ Coverage     33.93%   33.98%   +0.04%     
  Complexity     1105     1105              
============================================
  Files           189      189              
  Lines         10704    10712       +8     
  Branches       1744     1745       +1     
============================================
+ Hits           3632     3640       +8     
  Misses         6650     6650              
  Partials        422      422

@erikgb
Copy link
Contributor Author

erikgb commented Sep 12, 2019

@erikgb : Thanks, could you please add a line to CHANGELOG regarding this change?

@rohanKanojia : I have added a line in CHANGELOG about this change

@rohanKanojia rohanKanojia removed the pr/changelog-entry-please Please add a changelog entry for this PR label Sep 13, 2019
Copy link
Member

@rohanKanojia rohanKanojia left a comment

Choose a reason for hiding this comment

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

+1, Looks good. Thanks

@devang-gaur devang-gaur added the jkube/pending The issue/PR has to be taken care of in JKube https:/eclipse/jkube label Sep 19, 2019
@devang-gaur devang-gaur merged commit dddc3cb into fabric8io:master Sep 19, 2019
rohanKanojia added a commit to rohanKanojia/jshift that referenced this pull request Sep 26, 2019
@rohanKanojia rohanKanojia removed the jkube/pending The issue/PR has to be taken care of in JKube https:/eclipse/jkube label Sep 26, 2019
@erikgb erikgb deleted the pr/issue1708 branch October 4, 2019 16:23
rohanKanojia added a commit to rohanKanojia/jkube that referenced this pull request Nov 6, 2019
rohanKanojia added a commit to eclipse-jkube/jkube that referenced this pull request Nov 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants