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

Migrate from setup.py to pyproject #43

Merged
merged 3 commits into from
Oct 25, 2024

Conversation

cryptomilk
Copy link
Collaborator

Migrate from setup.py to pyproject.toml

@cryptomilk
Copy link
Collaborator Author

@dmulder do you prefer setuptools or pdm?

@dmulder
Copy link
Collaborator

dmulder commented Oct 22, 2024

@dmulder do you prefer setuptools or pdm?

I've never even heard of pdm. I have no preference.

@cryptomilk
Copy link
Collaborator Author

There is one issue I still haven't found out how to fix:

    data_files=[
        ('libexec/certmonger', ['bin/cepces-submit']),
    ],

I don't see how to do that, you can't seem to provide a destination to all files.

@cryptomilk cryptomilk force-pushed the asn-pyproject branch 2 times, most recently from ed83762 to bd27b11 Compare October 24, 2024 07:21
@cryptomilk
Copy link
Collaborator Author

I got it sorted out. I also added a tox.ini filed and use gh-actions for running tox now.

@cryptomilk cryptomilk changed the title Draft: Migrate from setup.py to pyproject Migrate from setup.py to pyproject Oct 24, 2024
@cryptomilk
Copy link
Collaborator Author

Check the tests output, it runs test for several python versions now.

@cryptomilk cryptomilk requested a review from dmulder October 25, 2024 06:12
@cryptomilk
Copy link
Collaborator Author

We could also enable linting and type checking if you want.

Copy link
Collaborator

@dmulder dmulder left a comment

Choose a reason for hiding this comment

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

LGTM!

@dmulder
Copy link
Collaborator

dmulder commented Oct 25, 2024

@cryptomilk I granted you write access to the repo, so feel free to merge when ready.

@cryptomilk cryptomilk merged commit ed7d87c into openSUSE:master Oct 25, 2024
5 checks passed
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