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

Make target for building wheel #1031

Merged
merged 3 commits into from Sep 30, 2021
Merged

Make target for building wheel #1031

merged 3 commits into from Sep 30, 2021

Conversation

smaragden
Copy link
Contributor

@smaragden smaragden commented Aug 3, 2021

Fixes #1005

Summary

Adds a wheel target to the Makefile.

Running make wheel will build a .whl package for the available python version to the root of the project.
Running make clean will remove all *.whl files from the project root.

@smaragden smaragden marked this pull request as ready for review August 4, 2021 14:37
Makefile Outdated Show resolved Hide resolved
@JeanChristopheMorinPerso
Copy link
Member

Thanks @smaragden for helping us on this! I just left a small comment regarding the command used to build the wheel.

@JeanChristopheMorinPerso
Copy link
Member

Oh, and if you want GitHub to automatically link the issue to this PR, you need to remove the back ticks around "Fixes #1005" in the PR description.

@smaragden
Copy link
Contributor Author

When merging, please squash the commits and use the Summary from the PR as message.

Replacing wheel build command from:
`python setup.py bdist_wheel` with:
`pip wheel . --no-deps`
This will have several advantages.

Use pyproject for dependencies. This makes it much cleaner to build
wheels.

We no longer need to check for the availability of the wheel package.
And the build will be out of source, and we don't need to cleanup
intermediate folders.

The `clean` target will now remove *.whl files in the root of the
project. This is where `make wheel` will output the wheels to with the
new command.
Copy link
Collaborator

@meshula meshula left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks!

@codecov-commenter
Copy link

Codecov Report

Merging #1031 (fb18ffc) into master (a84265c) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1031   +/-   ##
=======================================
  Coverage   79.30%   79.30%           
=======================================
  Files         113      113           
  Lines        6102     6102           
=======================================
  Hits         4839     4839           
  Misses       1263     1263           
Flag Coverage Δ
unittests 79.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a84265c...fb18ffc. Read the comment docs.

@ssteinbach ssteinbach added this to the Public Beta 14 milestone Aug 19, 2021
@ssteinbach ssteinbach added this to In progress in WG: Build Improvements via automation Aug 19, 2021
@ssteinbach ssteinbach merged commit 585df71 into AcademySoftwareFoundation:master Sep 30, 2021
WG: Build Improvements automation moved this from In progress to Done Sep 30, 2021
@ssteinbach
Copy link
Collaborator

Thank you @smaragden!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Add a make target to build a Python wheel locally
5 participants