Skip to content

Conversation

@rahmans1
Copy link
Contributor

@rahmans1 rahmans1 commented Sep 5, 2025

Briefly, what does this PR introduce?

Add action to trigger gitlab pipelines

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue #__)
  • Documentation update
  • Other: __

Please check if this PR fulfills the following:

  • Tests for the changes have been added
  • Documentation has been added / updated
  • Changes have been communicated to collaborators

Does this PR introduce breaking changes? What changes might users need to make to their code?

Does this PR change default behavior?

@rahmans1 rahmans1 requested review from veprbl and wdconinc September 5, 2025 14:58
Copy link
Contributor

@wdconinc wdconinc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you plan on continuing to use both eicweb and github, then you may need to make some more changes. I forget which ones, but the situation to avoid is:

  1. eicweb commit and push
  2. eicweb starts a pipeline
  3. eicweb does its mirroring push to github
  4. github actions sees a push event
  5. github actions triggers a second eicweb pipeline on the same commit

@rahmans1 rahmans1 merged commit 63a7f68 into main Sep 5, 2025
1 of 2 checks passed
@rahmans1 rahmans1 deleted the add-gitlab-mirroring branch September 8, 2025 12:27
@wdconinc wdconinc restored the add-gitlab-mirroring branch September 8, 2025 12:27
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.

3 participants