Skip to content

Modernize infrastructure to use pyproject.toml#3603

Merged
blnicho merged 7 commits intoPyomo:mainfrom
mrmundt:modernize-infrastructure
May 20, 2025
Merged

Modernize infrastructure to use pyproject.toml#3603
blnicho merged 7 commits intoPyomo:mainfrom
mrmundt:modernize-infrastructure

Conversation

@mrmundt
Copy link
Copy Markdown
Contributor

@mrmundt mrmundt commented May 14, 2025

Fixes NA

Summary/Motivation:

There is an impending removal of support for python setup.py develop being used in our infrastructure. This PR moves the minimum amount of information possible from setup.py to the more modern pyproject.toml. I also apparently have some black issues but I have no idea where they are coming from.

Changes proposed in this PR:

  • Introduce pyproject.toml
  • Remove setup.cfg
  • Make appropriate changes to setup.py to conform to PEP 517
  • Update docs
  • Update all workflows (tests and release)
  • Apply black for unknown reasons

NOTE: A passing Release distribution job is here: https://github.com/mrmundt/pyomo/actions/runs/15116644559

Legal Acknowledgement

By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the BSD license.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@mrmundt mrmundt requested review from blnicho and jsiirola May 14, 2025 17:19
Copy link
Copy Markdown
Member

@jsiirola jsiirola left a comment

Choose a reason for hiding this comment

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

Looks interesting. A couple questions, though...

Copy link
Copy Markdown
Member

@jsiirola jsiirola left a comment

Choose a reason for hiding this comment

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

Looks reasonable.

Copy link
Copy Markdown
Member

@blnicho blnicho left a comment

Choose a reason for hiding this comment

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

I have one minor question but otherwise this looks great. I also checked the RTD build and verified that this PR picks up the pyproject.toml file.

@blnicho blnicho merged commit 18def95 into Pyomo:main May 20, 2025
62 of 63 checks passed
@mrmundt mrmundt deleted the modernize-infrastructure branch May 20, 2025 15:06
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