Skip to content

feat: release v1.0.0 - #19

Merged
AlinsRan merged 1 commit into
mainfrom
release/0.20260727
Jul 28, 2026
Merged

feat: release v1.0.0#19
AlinsRan merged 1 commit into
mainfrom
release/0.20260727

Conversation

@AlinsRan

@AlinsRan AlinsRan commented Jul 27, 2026

Copy link
Copy Markdown

First release of this fork under semantic versioning, shipping #18 (fix(keys): reclaim expired shared-dict entries in remove_expired_keys()), which was merged after 0.20260716 was cut and is in no released version yet.

Why 1.0.0 (scheme change)

Earlier releases used the upstream date-based scheme (0.YYYYMMDD). Upstream (knyar) is dormant — last release 0.20240525 (2024-05), last commit 2025-01 — and this fork is now maintained independently, so versions no longer track upstream snapshots. Switching to semver aligns with every other Lua dependency in the apisix/EE rockspecs (all semver).

First semver release must be 1.0.0, not 0.x: luarocks compares by component, and 0.x would sort below the existing 0.2026xxxx date versions and be treated as a downgrade. Date-based tags before 1.0.0 are kept as-is.

Changes since 0.20260716

Release mechanism

Squash-merge keeping the title feat: release v1.0.0; the Release workflow matches ^feat: release v*(\S+), tags 1.0.0, and uploads the rockspec to luarocks. The workflow is version-format-agnostic, so no CI change is needed.

Downstream (after luarocks upload)

Summary by CodeRabbit

  • New Features
    • Released version 1.0.0 with a switch to semantic versioning.
  • Bug Fixes
    • Improved expired metric key cleanup so reclaimed shared-dictionary entries prevent per-worker metric key dictionaries from growing/shrinking improperly until full.
  • Chores
    • Updated package release metadata (rockspec and distribution config) to match version 1.0.0.

@coderabbitai

coderabbitai Bot commented Jul 27, 2026

Copy link
Copy Markdown

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

The changelog adds a 1.0.0 release entry and updates its commits URL. Package metadata and the rockspec adopt the API7 package name, semantic version, repository link, and 1.0.0 source tag.

Changes

Release metadata

Layer / File(s) Summary
Update release identifiers and changelog
CHANGELOG.md, dist.ini, rockspec/nginx-lua-prometheus-api7-1.0.0-1.rockspec
The changelog documents semantic versioning and expired-key reclamation, package metadata identifies the API7 package and repository, and the rockspec uses version 1.0.0-1 with source tag 1.0.0.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: releasing version 1.0.0.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
E2e Test Quality Review ✅ Passed Release-metadata only; no new runtime path is introduced, and the fix already has unit/integration coverage.
Security Check ✅ Passed Only release metadata/changelog changed; no logging, auth, storage, or TLS code paths were modified.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch release/0.20260727

Comment @coderabbitai help to get the list of available commands.

@AlinsRan
AlinsRan force-pushed the release/0.20260727 branch from 308e948 to c68ca4f Compare July 28, 2026 02:29
@AlinsRan AlinsRan changed the title feat: release v0.20260727 feat: release v0.20260728 Jul 28, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@rockspec/nginx-lua-prometheus-api7-0.20260728-1.rockspec`:
- Around line 4-8: Update the rockspec version and corresponding source tag in
the release metadata to the intended 0.20260727 release, ensuring the declared
filename/version matches the release workflow’s expected upload artifact. If
retaining 0.20260728 instead, update the release objective and commit metadata
and ensure that tag exists.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 35653dca-54c5-43ca-90e2-49822a5600af

📥 Commits

Reviewing files that changed from the base of the PR and between 308e948 and c68ca4f.

📒 Files selected for processing (1)
  • rockspec/nginx-lua-prometheus-api7-0.20260728-1.rockspec

Comment thread rockspec/nginx-lua-prometheus-api7-0.20260728-1.rockspec Outdated
@AlinsRan
AlinsRan force-pushed the release/0.20260727 branch from c68ca4f to 78780e3 Compare July 28, 2026 06:38
@AlinsRan AlinsRan changed the title feat: release v0.20260728 feat: release v1.0.0 Jul 28, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@CHANGELOG.md`:
- Around line 7-17: Synchronize the release identifiers to 0.20260728: update
CHANGELOG.md lines 7-17 from 1.0.0 to 0.20260728, update
rockspec/nginx-lua-prometheus-api7-1.0.0-1.rockspec lines 4-8 so the package
version and source tag use 0.20260728, and synchronize dist.ini before upload.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3717cbec-f3c0-4a5c-8088-d0af7c2caca2

📥 Commits

Reviewing files that changed from the base of the PR and between c68ca4f and 78780e3.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • rockspec/nginx-lua-prometheus-api7-1.0.0-1.rockspec

Comment thread CHANGELOG.md

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR prepares the first semantic-versioned release (v1.0.0) of the nginx-lua-prometheus-api7 fork, aligning release metadata (rockspec, dist config, and changelog) with the new versioning scheme and documenting the included fix from #18.

Changes:

  • Bump the LuaRocks rockspec version/tag to 1.0.0.
  • Update dist.ini version to 1.0.0.
  • Add a 1.0.0 section to the changelog describing the semver switch and #18.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
rockspec/nginx-lua-prometheus-api7-1.0.0-1.rockspec Updates LuaRocks package version/tag for the 1.0.0 release.
dist.ini Updates distribution metadata version to 1.0.0.
CHANGELOG.md Documents the 1.0.0 release and the semver scheme change.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread CHANGELOG.md
Comment thread dist.ini Outdated
@AlinsRan
AlinsRan force-pushed the release/0.20260727 branch from 13987a8 to 8bd0ea3 Compare July 28, 2026 06:56
@AlinsRan
AlinsRan force-pushed the release/0.20260727 branch from 8bd0ea3 to 493d70f Compare July 28, 2026 07:07
@AlinsRan
AlinsRan merged commit bc04f9d into main Jul 28, 2026
6 checks passed
@AlinsRan
AlinsRan deleted the release/0.20260727 branch July 28, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants