Skip to content

Conversation

@WillEngler
Copy link
Contributor

Summary

Adding the governance doc that Orion wrote up and everyone agreed to on the call. (I added it as-is, with one punctuation tweak to the first sentence in the UChicago/Argonne clause)

This also removes the extra legalese from contributing.md and removes the related checkbox from the PR template.

Checklist

Before a pull request can be merged, the following items must be checked:

  • Doc strings have been added in the Google docstring format.
  • Run ruff on your code.
  • Tests have been added for any new functionality or bug fixes.

We highly recommended installing the pre-commit hooks running in CI locally to speedup the development process. Simply run pip install pre-commit && pre-commit install to install the hooks which will check your code before each commit.

Copy link
Member

@CompRhys CompRhys left a comment

Choose a reason for hiding this comment

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

LGTM allowing for flaky test

@WillEngler WillEngler merged commit d35898b into TorchSim:main Oct 1, 2025
173 of 174 checks passed
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