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

python: move remaining projects to pyproject.toml #2905

Merged
merged 8 commits into from Apr 10, 2023

Conversation

tswhison
Copy link
Contributor

@tswhison tswhison commented Mar 29, 2023

Move all python projects to pyproject.toml projects. Removes the legacy behavior of calling python on setup.py, which is no longer the advised way of creating python packages. See https://peps.python.org/pep-0517/

Move packager to a pyproject.toml project. Removes the legacy
behavior of calling python on setup.py, which is no longer the
advised way of creating python packages.

Signed-off-by: Tim Whisonant <tim.whisonant@intel.com>
@tswhison tswhison self-assigned this Mar 29, 2023
@tswhison tswhison requested review from a team as code owners March 29, 2023 21:47
anandaravuri
anandaravuri previously approved these changes Mar 29, 2023
Signed-off-by: Tim Whisonant <tim.whisonant@intel.com>
anandaravuri
anandaravuri previously approved these changes Apr 3, 2023
anandaravuri
anandaravuri previously approved these changes Apr 4, 2023
@coveralls
Copy link

coveralls commented Apr 4, 2023

Pull Request Test Coverage Report for Build 4630681308

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 65.535%

Totals Coverage Status
Change from base Build 4612402913: 0.0%
Covered Lines: 13141
Relevant Lines: 20052

💛 - Coveralls

anandaravuri
anandaravuri previously approved these changes Apr 4, 2023
Signed-off-by: Tim Whisonant <tim.whisonant@intel.com>
@tswhison tswhison requested a review from a team as a code owner April 4, 2023 23:26
@tswhison tswhison changed the title packager: convert to modern pip package python: move remaining projects to pyproject.toml Apr 4, 2023
In support of legacy pip versions that don't respect pyproject.toml,
some portion of the setup.py file is still required.

Signed-off-by: Tim Whisonant <tim.whisonant@intel.com>
Signed-off-by: Tim Whisonant <tim.whisonant@intel.com>
@michael-adler
Copy link
Member

I checked that the venv generated by the OFS FIM build still works properly on SJ. Looks fine.

@tswhison
Copy link
Contributor Author

tswhison commented Apr 7, 2023

I plan to merge this to master on Monday so as not to break anything over the weekend.

@tswhison tswhison merged commit 12116f5 into feature/pip_install Apr 10, 2023
31 checks passed
@tswhison tswhison deleted the tswhison/pip_install branch April 10, 2023 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants