Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

move build config to pyproject.toml #1015

Draft
wants to merge 1 commit into
base: major-release-4
Choose a base branch
from

Conversation

bmos
Copy link
Contributor

@bmos bmos commented Mar 14, 2024

This is the new standard and it allows a lot of config to be done in this file, including configuration of dev tools like what I have suggested in #1013.

Doing this should also resolve the need for the LIMITED_DEPENDENCIES environment variable.
All packages have been grouped with an alias 'all' that installs everything but the dev requirements and tmc specific-packages.

I'm not sure about the Docker stuff though so I just made my best guess as to how to handle that.

It's also worth noting that LIMITED_DEPENDENCIES seems to have been implemented in such a way that most packages are not limited to a certain version, which means that you could end up with potentially quite different program behavior between LIMITED_DEPENDENCIES and normal install modes. This PR resolves this potential issue as all packages are declared per-module/in a single place.

@bmos bmos changed the base branch from main to major-release March 14, 2024 03:42
@bmos bmos marked this pull request as ready for review March 14, 2024 03:44
@jdw25 jdw25 mentioned this pull request May 17, 2024
@bmos bmos marked this pull request as draft June 25, 2024 22:29
@bmos
Copy link
Contributor Author

bmos commented Jun 29, 2024

I need to recreate this to take into account the intended behaviors of LIMITED_DEPENDENCIES

@shaunagm shaunagm changed the base branch from major-release to major-release-4 July 23, 2024 21:11
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