Skip to content

Auto sort#5

Open
Timple wants to merge 5 commits into
RoboStack:mainfrom
nobleo:auto-sort
Open

Auto sort#5
Timple wants to merge 5 commits into
RoboStack:mainfrom
nobleo:auto-sort

Conversation

@Timple

@Timple Timple commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

I vibe-coded some sorting scripts that keep the structure of the yaml files kind of aligned but with sorting enabled.

pixi run sort should help in the future.

After #4

@wep21 wep21 marked this pull request as draft April 18, 2026 12:01
@wep21

wep21 commented Apr 18, 2026

Copy link
Copy Markdown
Collaborator

I'm sorry but I've not finished initial build.

@Timple

Timple commented Apr 18, 2026

Copy link
Copy Markdown
Contributor Author

Is that related to sorting?

Or would you like this to happen on ros-kilted?
I picked rolling as that's usually where new stuff is happening :)

@Tobias-Fischer

Copy link
Copy Markdown
Contributor

@Timple can you please fix the merge conflicts?

@Tobias-Fischer Tobias-Fischer marked this pull request as ready for review July 3, 2026 09:40
@Tobias-Fischer

Copy link
Copy Markdown
Contributor

@wolfv @traversaro what do you think?

@Timple

Timple commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Rationale:

  • Easier to backport and forward port between ROS distros as the location of the dependency will be fixed.
  • Avoids merge conflicts because most contributors are inclined to append to the end of the list.

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.

3 participants