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

Seperate the dev CI tests to run for dev changes #307

Merged
merged 2 commits into from
Mar 18, 2024

Conversation

unkcpz
Copy link
Member

@unkcpz unkcpz commented Mar 18, 2024

Fixes #306

The CI action not triggered because the changes are made not from plugins.yaml file, thus not trigger the CI.
The following restriction is set to only trigger the preview CI to prevent the malicious code inject to CI actions.

paths:
- "plugins.yaml"

In this PR, I separate the actions that test changes on other development changes to other CI which will not term out to be a preview from our official aiida official domain. Besides, I remove the pre-commit action and make it triggered from GHA integration.

@unkcpz unkcpz force-pushed the seperate-user-test-and-dev-test branch 2 times, most recently from d476e3a to 570902e Compare March 18, 2024 11:22
unkcpz added 2 commits March 18, 2024 12:28
Fixes aiidateam#306

The CI action not triggered because the changes are made not from `plugins.yaml` file, thus not trigger the CI.
The following restriction is set to only trigger the preview CI to prevent the malicious code inject to CI actions.
https://github.com/aiidateam/aiida-registry/blob/c306d6a7cb85ef12eba51a24d4d3f21805e70b3d/.github/workflows/ci.yml#L10-L11

In this PR, I separate the actions that test changes on other development changes to other CI which will not term out to be a preview from our official aiida official domain. Besides, I remove the pre-commit action and make it triggered from GHA integration.
@unkcpz unkcpz force-pushed the seperate-user-test-and-dev-test branch from 8cb7adc to 258a0b6 Compare March 18, 2024 11:28
@unkcpz unkcpz merged commit 5644be4 into aiidateam:master Mar 18, 2024
2 of 3 checks passed
@unkcpz unkcpz deleted the seperate-user-test-and-dev-test branch March 18, 2024 11:28
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.

1 participant