Skip to content

Conversation

@palkerecsenyi
Copy link
Member

Closes inveniosoftware/invenio-github#188


  • We are updating invenio-github to be a generic package supporting many VCS providers, not just GitHub. As such, it will be renamed (currently to invenio-vcs).

  • The RDMGitHubRelease class has been renamed and restructured, so this must be reflected in config.py for continued functionality. No other changes are needed to this repository at the moment.

  • See compat for new VCS integration invenio-rdm-records#2128

@palkerecsenyi
Copy link
Member Author

Tests are currently failing due to a dependency issue which we need to fix.

We are importing and referencing RDMVCSRelease in the config. Currently this is failing as the invenio-vcs has not been published. However, we now want this to be an optional dependency rather than a default one (as it currently is in invenio-rdm-records, with the plugin being enabled through the GITHUB_INTEGRATION_ENABLED flag instead of the mere presence of the dependency). So we need to find a way to only apply the release class override if the user has manually installed the dependency and not to crash otherwise.

* We are updating `invenio-github` to be a generic package supporting
many VCS providers, not just GitHub. As such, it will be renamed
(currently to `invenio-vcs`).

* The `RDMGitHubRelease` class has been renamed and restructured, so
this must be reflected in `config.py` for continued functionality. No
other changes are needed to this repository at the moment.

* See inveniosoftware/invenio-rdm-records#2128
* Bumping the major version of invenio-oauthclient to 6.0.0

* This includes
inveniosoftware/invenio-oauthclient#360 which is
not necessarily breaking but requires a manual DB migration for very
large instances, as documented in the module's upgrade guide.

* Further changes are yet come in v6 before RDM v14, such as
inveniosoftware/invenio-oauthclient#328.
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.

Make invenio-github support other VCS providers

1 participant