Skip to content

Add pepsi format#2201

Merged
oleflb merged 8 commits intoHULKs:mainfrom
oleflb:pepsi-format
Mar 9, 2026
Merged

Add pepsi format#2201
oleflb merged 8 commits intoHULKs:mainfrom
oleflb:pepsi-format

Conversation

@oleflb
Copy link
Copy Markdown
Contributor

@oleflb oleflb commented Jan 25, 2026

Why? What?

Adds a pepsi format command, which runs cargo format, taplo fmt, and ruff format.
Right now the command does not expose any of the options of the three tools, not sure whether we want that.

The second commit also contains a format of the entire repo

Fixes #

ToDo / Known Issues

If this is a WIP describe which problems are to be fixed.

Ideas for Next Iterations (Not This PR)

  • integrate into CI

How to Test

  • pepsi fmt

@oleflb oleflb added the tools:Tooling Related to pepsi et.al. label Jan 25, 2026
@github-project-automation github-project-automation Bot moved this to Request for Review in Development Jan 25, 2026
@oleflb oleflb enabled auto-merge January 25, 2026 17:08
@knoellle knoellle self-assigned this Jan 27, 2026
@knoellle
Copy link
Copy Markdown
Member

For me, ./pepsi format fails with this error:

[ruff] ✗
   0: error: Failed to parse tools/machine-learning/mujoco/typings/mujoco/_specs.pyi:321:11: Expected an identifier
      error: Failed to parse tools/mujoco-simulator/mujoco-simulator/stubs/mujoco/_specs.pyi:321:11: Expected an identifier

Comment thread tools/pepsi/src/format.rs
Comment thread tools/pepsi/src/main.rs Outdated
@oleflb oleflb force-pushed the pepsi-format branch 2 times, most recently from 92b7602 to f1c3854 Compare March 9, 2026 16:40
Copy link
Copy Markdown
Member

@knoellle knoellle left a comment

Choose a reason for hiding this comment

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

LGTM

@oleflb oleflb added this pull request to the merge queue Mar 9, 2026
Merged via the queue into HULKs:main with commit 867b485 Mar 9, 2026
21 checks passed
@oleflb oleflb deleted the pepsi-format branch March 9, 2026 17:49
@github-project-automation github-project-automation Bot moved this from Request for Review to Done in Development Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tools:Tooling Related to pepsi et.al.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants