Skip to content

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
@poikilotherm poikilotherm deleted the release/v0.9.0 branch February 26, 2025 12:25
@poikilotherm
Copy link
Member

Replaced by new release workflow

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.

5 participants