Skip to content

Add two missing snippets#7023

Merged
Basilisk3 merged 1 commit intoFAForever:developfrom
Basilisk3:#7019_snippet
Feb 11, 2026
Merged

Add two missing snippets#7023
Basilisk3 merged 1 commit intoFAForever:developfrom
Basilisk3:#7019_snippet

Conversation

@Basilisk3
Copy link
Collaborator

@Basilisk3 Basilisk3 commented Feb 11, 2026

Description of the proposed changes

As per the title.

Summary by CodeRabbit

  • Balance Adjustments
    • Fire Beetle build time increased to 700.
    • Cybran T1 Frigate Proton Cannon range increased to 28.

@Basilisk3 Basilisk3 requested review from Rhaelya and lL1l1 February 11, 2026 20:52
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

This PR adds two changelog entries documenting balance adjustments: Fire Beetle build time increased from 550 to 700, and Trident Class frigate range restored from 26 to 28. Both entries are contextualized with rationale linking to related balance changes.

Changes

Cohort / File(s) Summary
Balance Changelog Entries
changelog/snippets/balance.7016.md, changelog/snippets/balance.7019.md
Documentation of two separate unit balance adjustments with economic and strategic rationale. Includes Fire Beetle build time revert and Trident Class range restoration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • Rhaelya

Poem

🐰 Hop, hop, the balance shifts today,
Beetles slower, frigates on their way,
Changelog whispers what the coders wrought,
Game more fair than once before 'twas fought! ⚔️

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is severely incomplete, only containing 'As per the title' without addressing required template sections. Complete the description by adding 'Testing done' and 'Additional context' sections, and ensure the checklist is completed as per the template.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add two missing snippets' accurately describes the PR's main objective—adding changelog snippet 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
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments
changelog/snippets/balance.7016.md (1)

1-1: Consider simplifying the phrasing.

The phrase "In conjunction with" could be replaced with a shorter alternative like "Combined with" or "Together with" for improved readability. As per coding guidelines, the static analysis tool suggests this is wordy.

✏️ Suggested simplification
-- (`#7016`) Increase Fire Beetle build time by reverting the reduction from `#5548`, since Beetles have shown to be too strong defensively and at the Tech 2 stage. In conjunction with the Deceiver nerf in `#6972`, the higher build time makes beetles require a larger infrastructure to build in large amounts for emergency defenses or for mixing into armies.
+- (`#7016`) Increase Fire Beetle build time by reverting the reduction from `#5548`, since Beetles have shown to be too strong defensively and at the Tech 2 stage. Combined with the Deceiver nerf in `#6972`, the higher build time makes beetles require a larger infrastructure to build in large amounts for emergency defenses or for mixing into armies.

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.

@Basilisk3 Basilisk3 enabled auto-merge (squash) February 11, 2026 20:55
@Basilisk3 Basilisk3 merged commit 44cf363 into FAForever:develop Feb 11, 2026
5 of 7 checks passed
@Basilisk3 Basilisk3 deleted the #7019_snippet branch February 11, 2026 22:41
@coderabbitai coderabbitai bot mentioned this pull request Feb 11, 2026
relent0r pushed a commit to relent0r/fa that referenced this pull request Feb 23, 2026
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