Skip to content

Update barricades.yml#252

Closed
Papiditel wants to merge 1 commit into
Civ13:masterfrom
Papiditel:Papiditel-patch-1
Closed

Update barricades.yml#252
Papiditel wants to merge 1 commit into
Civ13:masterfrom
Papiditel:Papiditel-patch-1

Conversation

@Papiditel
Copy link
Copy Markdown

Changelog

Media

@Papiditel Papiditel closed this Mar 11, 2026
@github-actions github-actions Bot added Untriaged This issue has not been reviewed yet Changes: No C# labels Mar 11, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 11, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8f769920-e54f-45da-b3fe-efe76ea949e4

📥 Commits

Reviewing files that changed from the base of the PR and between 459f0b9 and 8dd70f7.

📒 Files selected for processing (1)
  • Resources/Prototypes/Civ14/Entities/Structures/Walls/barricades.yml

Summary by CodeRabbit

Bug Fixes

  • Corrected asset references for multiple barricade structures (wooden, stone, sandstone, and brick variants) to ensure they display properly in the game.

Walkthrough

A sprite asset path correction across four barricade entity definitions, updating references from Civ14/Objects/Railing.rsi to Civ14/Objects/railing.rsi (capitalisation fix). No functional behaviour changes—purely asset reference updates.

Changes

Cohort / File(s) Summary
Sprite Asset Path Normalisation
Resources/Prototypes/Civ14/Entities/Structures/Walls/barricades.yml
Updated sprite references in four barricade entities (BarricadeRailingBase, BarricadeRailingStone, BarricadeRailingSandstone, BarricadeRailingBrick) from uppercase to lowercase path (Railing.rsirailing.rsi).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A railing once proud with a capital 'R',
Now humble and lowercase, like it should are,
Four barricades fixed with a simple correction,
Case sensitivity gets its affection! 🏰

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: No C# Untriaged This issue has not been reviewed yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant