You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Release 1.1.1 with the buffered-increment allocation fix from #28.
Update the rockspec version and source tag, distribution metadata, and changelog. Merging this release commit triggers the existing GitHub Release and LuaRocks publishing workflow.
This is a patch release: #28 changes no API or configuration.
Validation: the new rockspec passes luarocks lint.
Summary by CodeRabbit
Improvements
Counter increments and histogram observations now update pending values in place until synchronization, instead of creating a separate entry for every operation.
No actionable comments were generated in the recent review.
Walkthrough
This change adds a 1.1.1 changelog entry and updates the package version and rockspec version and source tag to 1.1.1.
Changes: Version 1.1.1 release
Layer / File(s)
Summary
Release notes and package version CHANGELOG.md, dist.ini, rockspec/nginx-lua-prometheus-api7-1.1.1-1.rockspec
The changelog records in-place updates to pending counter increments and histogram observations before sync(). The package version and rockspec version and source tag change to 1.1.1.
api7/nginx-lua-prometheus#27: Sets the preceding 1.1.0 release version in the same changelog and package metadata.
api7/nginx-lua-prometheus#28: Changes counter increments to update pending entries in place, as described in this release's changelog.
Suggested reviewers:shreemaan-abhishek
Merge Risk: Minimal · up to 7bd8e
The release changes are internally consistent and present no actionable merge risk.
Pre-merge checks: 6
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
The title clearly identifies the main change: the v1.1.1 release. The changes update the package metadata, rockspec, and changelog for that release.
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
E2e Test Quality Review
PASS. This PR changes only release metadata: CHANGELOG.md, dist.ini, and the versioned rockspec. It does not change application code or tests. E2E test completeness, assertions, mocking, error han…
Security Check
No security issues were introduced. Category 1: No sensitive data exposure. Category 2: No plaintext secret storage. Category 3: No endpoint or permission changes. Category 4: No cross-resource access…
Full details: E2e Test Quality Review
Explanation
PASS. This PR changes only release metadata: CHANGELOG.md, dist.ini, and the versioned rockspec. It does not change application code or tests. E2E test completeness, assertions, mocking, error handling, and concurrency are not applicable to this metadata-only release update. The version and source tag changes are consistent, and the existing release workflow already handles rockspec publishing.
Full details: Security Check
Explanation
No security issues were introduced. Category 1: No sensitive data exposure. Category 2: No plaintext secret storage. Category 3: No endpoint or permission changes. Category 4: No cross-resource access changes. Category 5: No TLS or cryptographic configuration changes. Category 6: No shared-resource operation changes. Category 7: No secret-reference handling changes. The diff only updates release metadata, the changelog, and the rockspec version/tag.
Finishing Touches
Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
CodeRabbit Verify
Run CodeRabbit Verify to test the changes in this pull request.
Recent review info
Run configuration
Source mode: Auto
Repository: Unavailable
Branch: Unavailable
File: No configuration file from this repository applied
Commit: Unavailable
Layers: Organization UI
Review profile: CHILL
Plan: Essentials
Run ID: 9cb5c42b-6dfb-4320-b7a2-3f51e33eec44
Commits
Reviewing files that changed from the base of the PR and between 869e6e0 and 7bd8e99.
Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release 1.1.1 with the buffered-increment allocation fix from #28.
Update the rockspec version and source tag, distribution metadata, and changelog. Merging this release commit triggers the existing GitHub Release and LuaRocks publishing workflow.
This is a patch release: #28 changes no API or configuration.
Validation: the new rockspec passes
luarocks lint.Summary by CodeRabbit