Skip to content

[release] Version 1.2.1#354

Merged
nemesifier merged 1 commit intomasterfrom
release/1.2.1
May 4, 2026
Merged

[release] Version 1.2.1#354
nemesifier merged 1 commit intomasterfrom
release/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 May 4, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9d2ce6d0-3e90-452f-a94c-276bd164e1a2

📥 Commits

Reviewing files that changed from the base of the PR and between 4a4c5fd and 73ecec8.

📒 Files selected for processing (2)
  • CHANGES.rst
  • rest_framework_gis/__init__.py
📜 Recent 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). (21)
  • GitHub Check: Python==3.12 | py312-django52-djangorestframework317
  • GitHub Check: Python==3.13 | py313-django52-djangorestframework317
  • GitHub Check: Python==3.12 | py312-django50-djangorestframework317
  • GitHub Check: Python==3.11 | py311-django52-djangorestframework316
  • GitHub Check: Python==3.11 | py311-django52-djangorestframework317
  • GitHub Check: Python==3.13 | py313-django52-djangorestframework316
  • GitHub Check: Python==3.13 | py313-django51-djangorestframework315
  • GitHub Check: Python==3.11 | py311-django51-djangorestframework315
  • GitHub Check: Python==3.11 | py311-django50-djangorestframework315
  • GitHub Check: Python==3.12 | py312-django52-djangorestframework315
  • GitHub Check: Python==3.12 | py312-django51-djangorestframework315
  • GitHub Check: Python==3.11 | py311-django51-djangorestframework316
  • GitHub Check: Python==3.10 | py310-django42-djangorestframework314
  • GitHub Check: Python==3.9 | py39-django42-djangorestframework314
  • GitHub Check: Python==3.12 | py312-django50-djangorestframework316
  • GitHub Check: Python==3.11 | py311-django50-djangorestframework316
  • GitHub Check: Python==3.12 | py312-django50-djangorestframework315
  • GitHub Check: Python==3.13 | py313-django52-djangorestframework315
  • GitHub Check: Analyze (actions)
  • GitHub Check: Analyze (python)
  • GitHub Check: Kilo Code Review
🧰 Additional context used
📓 Path-based instructions (5)
**/*.{js,ts,jsx,tsx,py,java,cs,go,rb,php,cpp,c,h,rs}

📄 CodeRabbit inference engine (Custom checks)

**/*.{js,ts,jsx,tsx,py,java,cs,go,rb,php,cpp,c,h,rs}: General: Flag potential security vulnerabilities
General: Flag obvious performance regressions such as heavy loops, repeated I/O, or unoptimized queries
General: Avoid unnecessary comments or docstrings for code that is already clear
General: Ensure code formatting is compact and readable; do not add excessive blank lines, especially inside function or method bodies
General: Flag unused or redundant code
General: Ensure variables, functions, classes, and files have descriptive and consistent names
General: Ensure new code handles errors properly - log errors that cannot be resolved by the user with error level
General: Log unusual conditions with warning level
General: Log important background actions with info level
General: Provide user-facing messages for errors that the user can solve autonomously (for example, validation errors)

Files:

  • rest_framework_gis/__init__.py
**/*.{js,ts,jsx,tsx,py,java,cs,go,rb,php,cpp,c,h,rs,sh,bash}

📄 CodeRabbit inference engine (Custom checks)

General: Cryptic or non-obvious code (regex, complex bash commands, or hard-to-read code) must include a concise comment explaining why it is needed and why the complexity is acceptable

Files:

  • rest_framework_gis/__init__.py
**/*.{md,rst,txt,adoc,js,ts,jsx,tsx,py,java,cs,go,rb,php,cpp,c,h,rs}

📄 CodeRabbit inference engine (Custom checks)

General: Flag outdated or incorrect documentation, comments, or docstrings

Files:

  • rest_framework_gis/__init__.py
  • CHANGES.rst
**/*.{py,html}

📄 CodeRabbit inference engine (Custom checks)

General (Django): Ensure all user-facing strings are marked as translatable using the Django i18n framework

Files:

  • rest_framework_gis/__init__.py
**/*.{md,rst,txt,adoc}

📄 CodeRabbit inference engine (Custom checks)

**/*.{md,rst,txt,adoc}: Changes: If the change affects documented behavior, update the documentation to reflect the new behavior
Features: Update documentation to mention the new feature; use a new documentation section or page if the feature is heavily UI-related

Files:

  • CHANGES.rst
🔇 Additional comments (2)
CHANGES.rst (1)

4-4: Changelog update looks correct and consistent with packaging metadata.

The 1.2.1 entry and DRF constraint note align with the declared dependency range in setup.py (djangorestframework>=3.12,<3.18).

Also applies to: 10-14

rest_framework_gis/__init__.py (1)

1-1: Version bump is correctly applied.

VERSION = (1, 2, 1, "final") is consistent with the release changelog and will flow into package metadata via get_version().


📝 Walkthrough

Walkthrough

This pull request releases version 1.2.1 of the package by updating the version constant in rest_framework_gis/__init__.py from (1, 2, 0, "final") to (1, 2, 1, "final"). The CHANGES.rst file is updated to mark this release with the date 2026-05-04 and record the Django REST Framework dependency constraint update to >=3.12,<3.18 with an associated issue reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title uses an unconventional [release] prefix instead of the required types (fix, change, feature, qa, ci, chores, docs), which does not conform to the specified format. Update the title to use one of the required type prefixes (e.g., [chores] Version 1.2.1) or clarify if [release] is an acceptable exception for release-related PRs.
Description check ❓ Inconclusive The description is minimal and does not follow the required template structure, missing sections like checklist, reference to issue, and detailed change description. While this is an auto-generated release PR, consider whether the template should be adapted for automated releases or if more details about the changes should be included.
✅ Passed checks (3 passed)
Check name Status Explanation
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.
Bug Fixes ✅ Passed This is a release PR that only updates version numbers and CHANGES.rst documentation, with no changes to core functionality or bug fixes.

✏️ 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 release/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
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

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

@nemesifier nemesifier self-assigned this May 4, 2026
@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented May 4, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

This is a standard release PR with version bump and changelog updates. All changes look correct.

Files Reviewed (2 files)
  • CHANGES.rst - Version date updated, formatting improvements, issue link added
  • rest_framework_gis/__init__.py - Version bumped from 1.2.0 to 1.2.1

Reviewed by kimi-k2.5-0127 · 47,492 tokens

@nemesifier nemesifier merged commit 0144f15 into master May 4, 2026
40 checks passed
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