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

Remove VCI and Security from the build actions #577

Merged
merged 2 commits into from
Sep 10, 2024

Conversation

kneckinator
Copy link
Contributor

@kneckinator kneckinator commented Sep 10, 2024

Why is this change needed?

As OpenG2P have moved the VCI addons from the openg2p-vci repo to the openg2-registry repo, we should no longer clone the openg2p-vci repo during the build.

The addons in the openg2p-security repo have also been moved to the openg2p-registry repo.

How was the change implemented?

The cloning of the openg2p-vci and openg2p-security repos have been removed.

Related links

OpenG2P-registry commit which adds the addons g2p_openid_vci_rest_api and g2p_openid_vci which were originally found in https://github.com/OpenG2P/openg2p-vci to https://github.com/OpenG2P/openg2p-registry

OpenG2P/openg2p-registry@ca2a73c

As OpenG2P have moved the VCI addons from the openg2p-vci repo to the openg2-registry repo, we should no longer clone the openg2p-vci repo during the build.
As OpenG2P have moved the Security addons from the openg2p-security repo to the openg2-registry repo, we should no longer clone the openg2p-registry repo during the build.
Copy link

sonarcloud bot commented Sep 10, 2024

@kneckinator kneckinator changed the title Remove VCI from the build actions Remove VCI and Security from the build actions Sep 10, 2024
@kneckinator kneckinator merged commit e48c22a into 17.0 Sep 10, 2024
6 of 8 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 10, 2024
@kneckinator kneckinator deleted the remove_vci_from_ci_job branch September 10, 2024 07:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant