Skip to content

[chores] Removed main branch from Backport workflow#1346

Merged
nemesifier merged 1 commit intoopenwisp:masterfrom
atif09:remove-main
May 7, 2026
Merged

[chores] Removed main branch from Backport workflow#1346
nemesifier merged 1 commit intoopenwisp:masterfrom
atif09:remove-main

Conversation

@atif09
Copy link
Copy Markdown
Contributor

@atif09 atif09 commented May 6, 2026

Checklist

  • I have read the OpenWISP Contributing Guidelines.
  • I have manually tested the changes proposed in this pull request.
  • I have written new test cases for new code and/or updated existing tests for changes to existing code.
  • I have updated the documentation.

Reference to Existing Issue

openwisp/openwisp-utils#501

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 6, 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: b2958ff0-f7e4-4596-a3b2-ac029ed9370e

📥 Commits

Reviewing files that changed from the base of the PR and between 0b202ff and dc1470a.

📒 Files selected for processing (1)
  • .github/workflows/backport.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/backport.yml
📜 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). (11)
  • GitHub Check: Python==3.11 | django~=5.2.0
  • GitHub Check: Python==3.10 | django~=5.1.0
  • GitHub Check: Python==3.12 | django~=5.1.0
  • GitHub Check: Python==3.11 | django~=4.2.0
  • GitHub Check: Python==3.12 | django~=5.2.0
  • GitHub Check: Python==3.13 | django~=5.1.0
  • GitHub Check: Python==3.10 | django~=5.2.0
  • GitHub Check: Python==3.13 | django~=5.2.0
  • GitHub Check: Python==3.12 | django~=4.2.0
  • GitHub Check: Python==3.11 | django~=5.1.0
  • GitHub Check: Python==3.10 | django~=4.2.0

📝 Walkthrough

Walkthrough

A GitHub Actions workflow file (.github/workflows/backport.yml) was modified to change the push trigger branch from main to master. No other workflow content was changed; the backport workflow will now run on pushes to master instead of main.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • nemesifier
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete, missing key sections including 'Description of Changes' and 'Screenshot', with only 2 of 4 checklist items addressed and no actual change description provided. Add a 'Description of Changes' section explaining why the main branch was removed, and complete the remaining checklist items and sections as applicable.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the required format [type] Descriptive title with 'chores' type and clearly describes the main change to the backport workflow.
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 PR is a GitHub Actions workflow configuration change, which falls under the valid exception for "fixes affecting Github Actions workflows." It does not affect core user-facing functionality.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Review ran into problems

🔥 Problems

Timed out fetching pipeline failures after 30000ms


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.

@kilo-code-bot
Copy link
Copy Markdown

kilo-code-bot Bot commented May 6, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The incremental diff shows the previous change was reverted - the main branch trigger has been added back to the workflow. The file now contains both master and main branch triggers, effectively restoring the original configuration.

Files Reviewed (1 file)
  • .github/workflows/backport.yml - Change reverted, both branch triggers present

Reviewed by kimi-k2.5-0127 · 91,883 tokens

@atif09 atif09 changed the title [chores] removed main [chores] Removed main from Backport workflow May 6, 2026
@atif09 atif09 changed the title [chores] Removed main from Backport workflow [chores] Removed main branch from Backport workflow May 6, 2026
@coveralls
Copy link
Copy Markdown

coveralls commented May 6, 2026

Coverage Status

coverage: 98.628%. remained the same — atif09:remove-main into openwisp:master

@nemesifier nemesifier merged commit 1a56617 into openwisp:master May 7, 2026
17 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.

4 participants