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

Port to pixi and QuantCo Oss copier template #153

Merged
merged 20 commits into from
Dec 17, 2024

Conversation

thomasmarwitz
Copy link
Contributor

@thomasmarwitz thomasmarwitz commented Dec 10, 2024

Checklist

  • Added a docs/changes.rst entry

@thomasmarwitz
Copy link
Contributor Author

thomasmarwitz commented Dec 10, 2024

@xhochy For now, I kept python 3.8. Or should we use this PR to remove support for 3.8 since it has already reached end of life:(https://devguide.python.org/versions/).

@xhochy
Copy link
Member

xhochy commented Dec 10, 2024

Please drop 3.8, we don't want to keep it as baggage.

Copy link

codecov bot commented Dec 10, 2024

Codecov Report

Attention: Patch coverage is 96.15385% with 2 lines in your changes missing coverage. Please review.

Project coverage is 85.38%. Comparing base (689e09f) to head (8106767).
Report is 29 commits behind head on main.

Files with missing lines Patch % Lines
minimalkv/_mixins.py 75.00% 1 Missing ⚠️
minimalkv/decorator.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            main     #153       +/-   ##
==========================================
+ Coverage   0.00%   85.38%   +85.38%     
==========================================
  Files         32       32               
  Lines       2001     2012       +11     
==========================================
+ Hits           0     1718     +1718     
+ Misses      2001      294     -1707     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@thomasmarwitz thomasmarwitz marked this pull request as ready for review December 13, 2024 09:48
@thomasmarwitz
Copy link
Contributor Author

Regarding version bump: Is this minor or patch?

name: artifact
path: dist/*

release:
Copy link
Contributor Author

@thomasmarwitz thomasmarwitz Dec 13, 2024

Choose a reason for hiding this comment

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

I have no way of testing this (the release workflow) but it basically just the copier template code

Comment on lines +48 to +49
user: __token__
password: ${{ secrets.pypi_password }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Except the authentication I added based on the previous ci.yml workflow

Copy link
Member

Choose a reason for hiding this comment

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

We should change this to OIDC instead of using a password.

I can do the testing once we have merge.

@thomasmarwitz
Copy link
Contributor Author

Lots of files were reformatted due to more pre-commit hooks. Also pyupgrade yielded some changes

@xhochy
Copy link
Member

xhochy commented Dec 16, 2024

Is this ready for a review?

@thomasmarwitz
Copy link
Contributor Author

Is this ready for a review?
Yes

@thomasmarwitz
Copy link
Contributor Author

@xhochy I added a changelog where I incremented the patch version.

@thomasmarwitz thomasmarwitz changed the title Port to pixi Port to pixi and QuantCo Oss copier template Dec 17, 2024
Copy link
Member

@xhochy xhochy left a comment

Choose a reason for hiding this comment

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

One comment, otherwise this is LGTM

.github/workflows/pre-commit-autoupdate.yml Outdated Show resolved Hide resolved
@xhochy xhochy merged commit 679b2fb into data-engineering-collective:main Dec 17, 2024
14 checks passed
@thomasmarwitz thomasmarwitz deleted the add-pixi branch December 17, 2024 11:44
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