Skip to content

Add refactoring ability for components - #525

Merged
alan412 merged 6 commits into
wpilibsuite:mainfrom
alan412:refactor_components
Sep 6, 2026
Merged

Add refactoring ability for components#525
alan412 merged 6 commits into
wpilibsuite:mainfrom
alan412:refactor_components

Conversation

@alan412

@alan412 alan412 commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Overview

This adds the ability to move components from the robot into either an existing or new mechanism.

This will allow the natural progression of starting with everything in a robot and moving them to a mechanism.

Linked Issues

Closes #524

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that causes existing functionality to change)
  • Documentation update
  • Code style/formatting updates
  • Refactoring / Renaming
  • Other

How Has This Been Tested?

  • Create component and move to new mechanism
  • Create component and move to existing mechanism with no name conflict
  • Create component and move to existing mechanism with name conflict
  • Move component that is used in robot methods
  • Move component that is used in open opmode methods and make sure it is renamed
  • Move component that is used in non-open opmodes and make sure it is renamed
  • Ran through testing checklist to make sure it didn't accidentally break something else.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • [N/A] I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have run through the [docs/testing_before_pr_checklist.md]

@alan412
alan412 requested a review from lizlooney August 28, 2026 16:31

@lizlooney lizlooney left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is really awesome work!

Comment thread frontend/blocks/mrc_call_python_function.ts
Comment thread frontend/blocks/mrc_call_python_function.ts
Comment thread frontend/blocks/mrc_component.ts
Comment thread frontend/blocks/mrc_call_python_function.ts Outdated
Comment thread frontend/editor/editor.ts Outdated
Comment thread frontend/editor/move_component_to_mechanism.ts
@alan412
alan412 requested a review from lizlooney August 31, 2026 14:55
@alan412
alan412 merged commit 7960a4e into wpilibsuite:main Sep 6, 2026
1 check passed
@alan412
alan412 deleted the refactor_components branch September 6, 2026 11:07
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.

Need way to easily refactor components into mechanisms

2 participants