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

Release v0.9.0 #304

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

Release v0.9.0 #304

wants to merge 2 commits into from

Conversation

sdruskat
Copy link
Contributor

Prepares hermes for the 0.9.0 release.

@sdruskat
Copy link
Contributor Author

Current test coverage is < 65%, see CONTRIBUTING.md:

1. Ensure test coverage is at least 65%, and that all tests pass.

---------- coverage: platform linux, python 3.12.3-final-0 -----------
Name                                                    Stmts   Miss  Cover
---------------------------------------------------------------------------
src/hermes/__init__.py                                      0      0   100%
src/hermes/__main__.py                                      3      0   100%
src/hermes/commands/__init__.py                             8      0   100%
src/hermes/commands/base.py                                70     13    81%
src/hermes/commands/clean/__init__.py                       0      0   100%
src/hermes/commands/clean/base.py                          12      2    83%
src/hermes/commands/cli.py                                 37      7    81%
src/hermes/commands/curate/__init__.py                      0      0   100%
src/hermes/commands/curate/base.py                         23      8    65%
src/hermes/commands/deposit/__init__.py                     0      0   100%
src/hermes/commands/deposit/base.py                        73     41    44%
src/hermes/commands/deposit/error.py                        2      0   100%
src/hermes/commands/deposit/file.py                        15      5    67%
src/hermes/commands/deposit/invenio.py                    284    205    28%
src/hermes/commands/deposit/invenio_rdm.py                 47     29    38%
src/hermes/commands/deposit/rodare.py                      31     11    65%
src/hermes/commands/harvest/__init__.py                     0      0   100%
src/hermes/commands/harvest/base.py                        34     15    56%
src/hermes/commands/harvest/cff.py                         73     49    33%
src/hermes/commands/harvest/codemeta.py                    31     20    35%
src/hermes/commands/harvest/util/__init__.py                0      0   100%
src/hermes/commands/harvest/util/validate_codemeta.py      31     26    16%
src/hermes/commands/init/__init__.py                        0      0   100%
src/hermes/commands/init/base.py                          388    296    24%
src/hermes/commands/init/connect_github.py                 59     40    32%
src/hermes/commands/init/connect_gitlab.py                 87     70    20%
src/hermes/commands/init/connect_zenodo.py                 45     18    60%
src/hermes/commands/init/oauth_process.py                 154    119    23%
src/hermes/commands/init/slim_click.py                     81     52    36%
src/hermes/commands/postprocess/__init__.py                 0      0   100%
src/hermes/commands/postprocess/base.py                    12      1    92%
src/hermes/commands/postprocess/invenio.py                 28     21    25%
src/hermes/commands/postprocess/invenio_rdm.py             14      9    36%
src/hermes/commands/process/__init__.py                     0      0   100%
src/hermes/commands/process/base.py                        42     15    64%
src/hermes/error.py                                         2      0   100%
src/hermes/logger.py                                       18      0   100%
src/hermes/model/__init__.py                                0      0   100%
src/hermes/model/context.py                               195     77    61%
src/hermes/model/errors.py                                 11      5    55%
src/hermes/model/merge.py                                 107     57    47%
src/hermes/model/path.py                                  216     89    59%
src/hermes/utils.py                                         7      0   100%
---------------------------------------------------------------------------
TOTAL                                                    2240   1300    42%

@led02
Copy link
Member

led02 commented Jan 11, 2025

Okay, looking at the distribution of the test coverage, it's way less the parts that I would have to take the shame for...

I'll have a look at what I can do to get this ready for merge :)

@nheeb nheeb marked this pull request as ready for review January 13, 2025 10:23
Comment on lines +57 to +58
- given-names: Heeb
family-names: Nitai
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- given-names: Heeb
family-names: Nitai
- given-names: Nitai
family-names: Heeb

@nheeb nheeb marked this pull request as draft January 13, 2025 10:30
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.

3 participants