Skip to content

feat: release v7.1.1#12

Merged
nic-6443 merged 1 commit intomasterfrom
nic/7.1.1
Apr 1, 2026
Merged

feat: release v7.1.1#12
nic-6443 merged 1 commit intomasterfrom
nic/7.1.1

Conversation

@jarvis9443
Copy link
Copy Markdown

@jarvis9443 jarvis9443 commented Apr 1, 2026

Signed-off-by: Jarvis jarvis@api7.ai

Summary by CodeRabbit

Release Notes - Version 7.1.1

  • Bug Fixes

    • Fixed a caching issue where unnecessary records were included in DNS query results.
  • Chores

    • Added build configuration for version 7.1.1 release.

Signed-off-by: Jarvis <jarvis@api7.ai>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

📝 Walkthrough

Walkthrough

Version 7.1.1 release is documented through two additions: a changelog entry noting a bug fix for finalCacheOnly result filtering, and a corresponding LuaRocks rockspec file defining package metadata, dependencies, and module mappings for the release.

Changes

Cohort / File(s) Summary
Release Documentation
README.md
Added changelog entry for version 7.1.1 (01-Apr-2026) documenting a fix to exclude additional section records from finalCacheOnly results, linked to PR #11.
Package Configuration
rockspec/api7-lua-resty-dns-client-7.1.1-0.rockspec
New LuaRocks rockspec file defining package metadata, Apache 2.0 license, runtime dependencies, and Lua module mappings (resty.dns.*) for the 7.1.1-0 release.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Poem

🐰 A changelog line, a rockspec bright,
Version seven point one point one takes flight,
Cache records pruned with DNS delight,
Hoppy release day, everything's right! 🎉

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The pull request title 'feat: release v7.1.1' accurately summarizes the main change - creating a release for version 7.1.1 by adding changelog and rockspec files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch nic/7.1.1

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

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
rockspec/api7-lua-resty-dns-client-7.1.1-0.rockspec (1)

4-4: Prefer HTTPS for rockspec source URL.

At Line 4, git:// can be blocked in some environments and is less robust than HTTPS for fetches.

Proposed change
-    url = "git://github.com/api7/lua-resty-dns-client",
+    url = "https://github.com/api7/lua-resty-dns-client.git",
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@rockspec/api7-lua-resty-dns-client-7.1.1-0.rockspec` at line 4, The
rockspec's source URL uses the git protocol; update the url assignment (the url
field in the rockspec containing "git://github.com/api7/lua-resty-dns-client")
to use an HTTPS Git URL (https://github.com/api7/lua-resty-dns-client.git) so
fetches work in restricted environments and are more robust.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 56: The new release heading "### 7.1.1 (01-Apr-2026)" violates
markdownlint rules MD001/MD003; update this heading to match the repository's
existing release heading level and style (use the same atx/hash count and
punctuation/spacing as other release headings in README.md) so it is consistent
with the file's other release entries and passes linting.

---

Nitpick comments:
In `@rockspec/api7-lua-resty-dns-client-7.1.1-0.rockspec`:
- Line 4: The rockspec's source URL uses the git protocol; update the url
assignment (the url field in the rockspec containing
"git://github.com/api7/lua-resty-dns-client") to use an HTTPS Git URL
(https://github.com/api7/lua-resty-dns-client.git) so fetches work in restricted
environments and are more robust.
🪄 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: 0f002396-ac06-4505-b402-fd2dcdc312c5

📥 Commits

Reviewing files that changed from the base of the PR and between b9e4d9a and 163d990.

📒 Files selected for processing (2)
  • README.md
  • rockspec/api7-lua-resty-dns-client-7.1.1-0.rockspec

4. commit and tag the release
5. upload rock to LuaRocks

### 7.1.1 (01-Apr-2026)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Align the new release heading with repository markdownlint rules.

At Line 56, the new ### heading triggers both MD001 and MD003 in static analysis. Please normalize this heading to the expected level/style used by your lint config so release docs stay lint-clean.

Proposed doc-only adjustment
-### 7.1.1 (01-Apr-2026)
+7.1.1 (01-Apr-2026)
+-------------------
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
### 7.1.1 (01-Apr-2026)
7.1.1 (01-Apr-2026)
-------------------
🧰 Tools
🪛 markdownlint-cli2 (0.22.0)

[warning] 56-56: Heading levels should only increment by one level at a time
Expected: h2; Actual: h3

(MD001, heading-increment)


[warning] 56-56: Heading style
Expected: setext; Actual: atx

(MD003, heading-style)

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` at line 56, The new release heading "### 7.1.1 (01-Apr-2026)"
violates markdownlint rules MD001/MD003; update this heading to match the
repository's existing release heading level and style (use the same atx/hash
count and punctuation/spacing as other release headings in README.md) so it is
consistent with the file's other release entries and passes linting.

@nic-6443 nic-6443 merged commit 0449c22 into master Apr 1, 2026
2 of 3 checks passed
@nic-6443 nic-6443 deleted the nic/7.1.1 branch April 1, 2026 09:00
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.

2 participants