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

CI: Build with build #49

Closed
wants to merge 3 commits into from
Closed

CI: Build with build #49

wants to merge 3 commits into from

Conversation

effigies
Copy link
Contributor

Calling python setup.py directly is deprecated. Using a dedicated build tool like build is the thing now. Also switched to pipx to bypass the install step; pipx will ensure the tools we directly use get installed and build will ensure the build-time dependencies get installed.

@codecov
Copy link

codecov bot commented Oct 13, 2023

Codecov Report

Merging #49 (70c27eb) into master (0d7dbe2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #49   +/-   ##
=======================================
  Coverage   83.56%   83.56%           
=======================================
  Files           3        3           
  Lines          73       73           
=======================================
  Hits           61       61           
  Misses         12       12           
Flag Coverage Δ
unittests 83.56% <ø> (ø)

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

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@effigies effigies force-pushed the mnt/build branch 2 times, most recently from 303b8fd to 581b9fa Compare October 13, 2023 15:14
@yarikoptic
Copy link
Collaborator

I trust you on this @effigies ! but now we need a rebase due to dependabot changes have been merged

@effigies
Copy link
Contributor Author

Would it make sense to run this on all PR/push and just condition the publish step on release?

@effigies
Copy link
Contributor Author

Did it anyway. Now no need to trust.

@effigies effigies closed this Nov 13, 2023
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