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

Use PEP-621 for project metadata, extras for deps and VCS for version #12

Merged
merged 3 commits into from
May 11, 2022
Merged

Use PEP-621 for project metadata, extras for deps and VCS for version #12

merged 3 commits into from
May 11, 2022

Conversation

gaborbernat
Copy link
Contributor

Signed-off-by: Bernát Gábor bgabor8@bloomberg.net

@gaborbernat gaborbernat changed the title test Use PEP-621 for project metadata, extras for deps and VCS for version May 11, 2022
bumpversion is no longer used.

Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
Makefile Show resolved Hide resolved
@pablogsal
Copy link
Member

Hummm, seems that either we were not checking correctly coverage or for some reason the coverage went down :(

@gaborbernat
Copy link
Contributor Author

We were not checking it correctly, looked at coverage HTML report and looked legit.

@pablogsal
Copy link
Member

We were not checking it correctly, looked at coverage HTML report and looked legit.

Then decrease the percentage so the check passes, please

Signed-off-by: Bernát Gábor <bgabor8@bloomberg.net>
@pablogsal pablogsal merged commit 6d2c69b into bloomberg:main May 11, 2022
@gaborbernat gaborbernat deleted the docker branch May 11, 2022 15:58
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.

None yet

2 participants