Skip to content

[docs] Port changelog for release 1.2.1#253

Merged
nemesifier merged 1 commit intomasterfrom
chore/port-changelog-1.2.1
Apr 9, 2026
Merged

[docs] Port changelog for release 1.2.1#253
nemesifier merged 1 commit intomasterfrom
chore/port-changelog-1.2.1

Conversation

@nemesifier
Copy link
Copy Markdown
Member

Note: This PR was automatically generated by the openwisp-utils releaser tool.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 9, 2026

📝 Walkthrough

Walkthrough

The pull request adds a changelog entry for version 1.2.1 dated 2026-04-09 to the CHANGELOG.rst file. The entry documents a bugfix under a Bugfixes subsection that addresses UCI name escaping in the uci-autoname script. No other changelog content, versions, or code files were modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is minimal but appropriate for an auto-generated changelog PR; however, it lacks the required template sections including checklist items, issue reference, and detailed change description. Consider whether template compliance is required for automated tool-generated PRs. If required, add standard template sections; if not, clarify policies for auto-generated PRs.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the required format [docs] and accurately describes the changelog entry addition for release 1.2.1.
Bug Fixes ✅ Passed This PR only modifies CHANGELOG.rst to document a previously implemented bug fix, not a code fix PR.

✏️ 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 chore/port-changelog-1.2.1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@CHANGELOG.rst`:
- Line 10: Update the changelog entry text to reference the actual script name
used in the codebase: replace "uci-autoname script" with "openwisp-uci-autoname
script" so the line reads "Fixed UCI name escaping in openwisp-uci-autoname
script"; locate the string in CHANGELOG.rst and edit that entry to match the
filename and installation references
(openwisp-config/files/sbin/openwisp-uci-autoname and
$(1)/usr/sbin/openwisp-uci-autoname) and keep formatting consistent with other
entries.
🪄 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: ASSERTIVE

Plan: Pro

Run ID: 70f75e31-e904-4df2-ba8a-705469529c89

📥 Commits

Reviewing files that changed from the base of the PR and between d3b6ae1 and f1f4324.

📒 Files selected for processing (1)
  • CHANGELOG.rst
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Analyze (actions)
  • GitHub Check: Kilo Code Review

Comment thread CHANGELOG.rst
Bugfixes
~~~~~~~~

- Fixed UCI name escaping in uci-autoname script
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

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Description: Find the actual script name in the codebase to verify changelog accuracy

# Search for script files with 'autoname' in the name
echo "=== Searching for autoname scripts ==="
fd -t f -e sh 'autoname'

# Search for references to both script names in the codebase
echo -e "\n=== Searching for 'openwisp-uci-autoname' references ==="
rg -l 'openwisp-uci-autoname' --type sh

echo -e "\n=== Searching for 'uci-autoname' references (excluding openwisp- prefix) ==="
rg -l '\buci-autoname\b' --type sh -g '!CHANGELOG.rst'

Repository: openwisp/openwisp-config

Length of output: 242


🏁 Script executed:

#!/bin/bash
# Broader search to find script references

echo "=== Checking for scripts in common locations ==="
find . -type f \( -name "*autoname*" -o -name "bin" -o -name "scripts" \) 2>/dev/null | head -20

echo -e "\n=== Searching all file types for 'openwisp-uci-autoname' ==="
rg 'openwisp-uci-autoname' --type-list | head -5
rg 'openwisp-uci-autoname' -g '!.git' 2>/dev/null | head -10

echo -e "\n=== Searching all file types for 'uci-autoname' (broader match) ==="
rg 'uci-autoname' -g '!.git' -g '!CHANGELOG.rst' 2>/dev/null | head -10

echo -e "\n=== Checking setup.py or package metadata ==="
fd -t f '(setup|pyproject|setup\.cfg)' | xargs cat 2>/dev/null | grep -A 5 -B 5 'autoname' 2>/dev/null

echo -e "\n=== Checking directory structure ==="
ls -la | grep -E '^d'
find . -maxdepth 2 -type d -name '*bin*' -o -name '*script*' 2>/dev/null

Repository: openwisp/openwisp-config

Length of output: 2061


Fix the script name to match codebase references.

The changelog entry refers to "uci-autoname script", but the actual script in the codebase is named "openwisp-uci-autoname" (with the openwisp- prefix). This is evident from the script location (openwisp-config/files/sbin/openwisp-uci-autoname.lua), Makefile installation ($(1)/usr/sbin/openwisp-uci-autoname), and previous changelog entries (lines 177, 190) which all reference "openwisp-uci-autoname". Update the entry to: "Fixed UCI name escaping in openwisp-uci-autoname script"

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

In `@CHANGELOG.rst` at line 10, Update the changelog entry text to reference the
actual script name used in the codebase: replace "uci-autoname script" with
"openwisp-uci-autoname script" so the line reads "Fixed UCI name escaping in
openwisp-uci-autoname script"; locate the string in CHANGELOG.rst and edit that
entry to match the filename and installation references
(openwisp-config/files/sbin/openwisp-uci-autoname and
$(1)/usr/sbin/openwisp-uci-autoname) and keep formatting consistent with other
entries.

@nemesifier nemesifier merged commit 250c5d1 into master Apr 9, 2026
5 of 6 checks passed
@nemesifier nemesifier deleted the chore/port-changelog-1.2.1 branch April 9, 2026 17:34
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.

1 participant