-
Notifications
You must be signed in to change notification settings - Fork 3.2k
fix(deps): update module github.com/nginxinc/nginx-prometheus-exporter to v1 #31071
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
Conversation
4fd4097 to
6ee8cfd
Compare
d0f7d0f to
1c679c8
Compare
1c679c8 to
54682cf
Compare
54682cf to
1a72f33
Compare
1a72f33 to
f459ac5
Compare
f459ac5 to
ee1223e
Compare
c05089a to
c784a0c
Compare
c784a0c to
7753b00
Compare
codeboten
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pinging code owner: @djaglowski
8c68567 to
e463f5d
Compare
e463f5d to
799d989
Compare
|
CI fails with |
799d989 to
d6e0b2d
Compare
|
This PR was marked stale due to lack of activity. It will be closed in 14 days. |
|
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
This change has had a few previous attempts, mostly by renovate, which largely failed due to the rename of the upstream's org from `nginxinc` to `nginx`. In addition, the function signature for the Nginx client initialization changed in v1 to no longer return an error. To account for that inspiration was taken from open-telemetry#31071 to move the client constructor to the `start` function. This is my first attempt at contributing to this project, so please let me know if I need to adjust anything. Thank you. Related to: open-telemetry#38545 open-telemetry#36999 open-telemetry#31272 open-telemetry#31071
This change has had a few previous attempts, mostly by renovate, which largely failed due to the rename of the upstream's org from `nginxinc` to `nginx`. In addition, the function signature for the Nginx client initialization changed in v1 to no longer return an error. To account for that inspiration was taken from open-telemetry#31071 to move the client constructor to the `start` function. This is my first attempt at contributing to this project, so please let me know if I need to adjust anything. Thank you. Related to: open-telemetry#38545 open-telemetry#36999 open-telemetry#31272 open-telemetry#31071
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> #### Description This change has had a few previous attempts, mostly by renovate, which largely failed due to the rename of the upstream's org from `nginxinc` to `nginx`. In addition, the function signature for the Nginx client initialization changed in v1 to no longer return an error. To account for that inspiration was taken from #31071 to move the client constructor to the `start` function. This is my first attempt at contributing to this project, so please let me know if I need to adjust anything. Thank you. <!-- Issue number (e.g. #1234) or full URL to issue, if applicable. --> #### Link to tracking issue No open issues, but related to these previous PRs: #38545 #36999 #31272 #31071. <!--Describe what testing was performed and which tests were added.--> #### Testing Ran tests via `make` locally. <!--Describe the documentation added.--> #### Documentation n/a <!--Please delete paragraphs that you did not use before submitting.-->
…elemetry#38799) <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> #### Description This change has had a few previous attempts, mostly by renovate, which largely failed due to the rename of the upstream's org from `nginxinc` to `nginx`. In addition, the function signature for the Nginx client initialization changed in v1 to no longer return an error. To account for that inspiration was taken from open-telemetry#31071 to move the client constructor to the `start` function. This is my first attempt at contributing to this project, so please let me know if I need to adjust anything. Thank you. <!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. --> #### Link to tracking issue No open issues, but related to these previous PRs: open-telemetry#38545 open-telemetry#36999 open-telemetry#31272 open-telemetry#31071. <!--Describe what testing was performed and which tests were added.--> #### Testing Ran tests via `make` locally. <!--Describe the documentation added.--> #### Documentation n/a <!--Please delete paragraphs that you did not use before submitting.-->
…elemetry#38799) <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> #### Description This change has had a few previous attempts, mostly by renovate, which largely failed due to the rename of the upstream's org from `nginxinc` to `nginx`. In addition, the function signature for the Nginx client initialization changed in v1 to no longer return an error. To account for that inspiration was taken from open-telemetry#31071 to move the client constructor to the `start` function. This is my first attempt at contributing to this project, so please let me know if I need to adjust anything. Thank you. <!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. --> #### Link to tracking issue No open issues, but related to these previous PRs: open-telemetry#38545 open-telemetry#36999 open-telemetry#31272 open-telemetry#31071. <!--Describe what testing was performed and which tests were added.--> #### Testing Ran tests via `make` locally. <!--Describe the documentation added.--> #### Documentation n/a <!--Please delete paragraphs that you did not use before submitting.-->
This PR contains the following updates:
v0.11.0->v1.0.0Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Release Notes
nginxinc/nginx-prometheus-exporter (github.com/nginxinc/nginx-prometheus-exporter)
v1.0.0Compare Source
What's Changed
💣 Breaking Changes
🚀 Features
🐛 Bug Fixes
🔨 Maintenance
33 changes
actions/labelerby @lucacome in https://github.com/nginxinc/nginx-prometheus-exporter/pull/295📝 Documentation
⬆️ Dependencies
191 changes
0861fa5to4c74e84by @dependabot in https://github.com/nginxinc/nginx-prometheus-exporter/pull/2984c74e84to7b1327bby @dependabot in https://github.com/nginxinc/nginx-prometheus-exporter/pull/344New Contributors
Configuration
📅 Schedule: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.