Skip to content

Fix bundle.yaml layout spacing to use consistent 2-space indentation#3003

Closed
Copilot wants to merge 3 commits intolocalize-panelsfrom
copilot/sub-pr-3002
Closed

Fix bundle.yaml layout spacing to use consistent 2-space indentation#3003
Copilot wants to merge 3 commits intolocalize-panelsfrom
copilot/sub-pr-3002

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 10, 2026

Addresses feedback from #3002 to ensure consistent spacing in bundle.yaml layout sections across all pyRevit extensions.

Changes

  • Fixed Wipe.pulldown/bundle.yaml layout section to use 2 spaces before list items (was missing indentation)
  • Verified all other bundle.yaml files already conform to the 2-space indentation standard

Before:

layout:
- Wipe Model Components
- Wipe Attached Constraints

After:

layout:
  - Wipe Model Components
  - Wipe Attached Constraints

This maintains consistency with pyRevit's bundle.yaml formatting convention used throughout the codebase.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copy link
Copy Markdown
Contributor

@devloai devloai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary:

This PR was created to address feedback on PR #3002 regarding YAML formatting in bundle.yaml files. Specifically, it aims to ensure consistent use of spaces (2-space indent) before list item dashes in bundle.yaml files. Currently, the PR is marked as [WIP] and contains no file changes.

Review Summary:

No review comments were recorded as this PR currently has 0 additions, 0 deletions, and 0 changed files. The PR appears to be a placeholder created by the Copilot bot in response to comment #2678800794 which requested: "always use spaces when list items like this in bundle.yaml files" for sections like layout: that contain YAML list items.

The expected changes would convert YAML lists from:

layout:
- Edit
- Override

to:

layout:
  - Edit
  - Override

Follow-up suggestions:

  • Close this PR if the YAML spacing fixes will be applied directly to PR #3002
  • Commit the formatting changes to make this PR ready for review if it should remain a separate sub-PR

…tems

Co-authored-by: jmcouffin <7872003+jmcouffin@users.noreply.github.com>
Copilot AI changed the title [WIP] Update localization panel for bundle.yaml spacing Fix bundle.yaml layout spacing to use consistent 2-space indentation Jan 10, 2026
Copilot AI requested a review from jmcouffin January 10, 2026 16:35
@jmcouffin jmcouffin closed this Jan 10, 2026
@jmcouffin jmcouffin deleted the copilot/sub-pr-3002 branch January 10, 2026 16:38
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