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

Fix ICLA Check #944

Merged
merged 1 commit into from
Oct 10, 2024
Merged

Fix ICLA Check #944

merged 1 commit into from
Oct 10, 2024

Conversation

Arnei
Copy link
Member

@Arnei Arnei commented Oct 10, 2024

Copied from opencast/opencast#6238

The tool we use for checking if contributors signed their ICLA is failing to install on CI runs with new Python versions since we just install it system wide. While that may not be good practice on your local machine, this shouldn't matter in the CI since we immediately throw away the machine anyway.

Copied from opencast/opencast#6238

The tool we use for checking if contributors signed their ICLA is
failing to install on CI runs with new Python versions since we just
install it system wide. While that may not be good practice on your
local machine, this shouldn't matter in the CI since we immediately
throw away the machine anyway.
Copy link
Contributor

Use docker or podman to test this pull request locally.

Run test server using develop.opencast.org as backend:

podman run --rm -it -p 127.0.0.1:3000:3000 ghcr.io/opencast/opencast-admin-interface:pr-944

Specify a different backend like stable.opencast.org:

podman run --rm -it -p 127.0.0.1:3000:3000 -e PROXY_TARGET=https://stable.opencast.org ghcr.io/opencast/opencast-admin-interface:pr-944

It may take a few seconds for the interface to spin up.
It will then be available at http://127.0.0.1:3000.
For more options you can pass on to the proxy, take a look at the README.md.

Copy link
Contributor

This pull request is deployed at test.admin-interface.opencast.org/944/2024-10-10_08-40-07/ .
It might take a few minutes for it to become available.

@opencast opencast deleted a comment from github-actions bot Oct 10, 2024
@lkiesow
Copy link
Member

lkiesow commented Oct 10, 2024

Makes sense to get this in as soon as possible → just merging this

@lkiesow lkiesow merged commit 5bbc4fb into opencast:main Oct 10, 2024
4 of 6 checks passed
@lkiesow lkiesow self-assigned this Oct 10, 2024
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.

2 participants