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

add ci tests for gitlab integration #119

Merged
merged 5 commits into from
Nov 30, 2020
Merged

Conversation

preslavmihaylov
Copy link
Owner

@preslavmihaylov preslavmihaylov commented Nov 29, 2020

Related to #120

One of the CI steps fail because the new secret is not available for pull_request_target runs. As observed, when the event is push, the step passes. After this is merged, the failing step will, hence, start passing.

In addition to this, I've decided to reduce the test verbosity to more easily identify which tests are failing.

Closes #117

@mehdy
Copy link
Collaborator

mehdy commented Nov 30, 2020

@preslavmihaylov The tests are still failing.

@preslavmihaylov
Copy link
Owner Author

@preslavmihaylov The tests are still failing.

Yeah, I've mentioned that in the description - tests whichare run with pull_request_target run with the workflow as-is from the master branch. The workflow from there doesn't have the new secret I've added.

After the diff is merged, the test will start passing, as is seen by the push variant of the step.

@mehdy
Copy link
Collaborator

mehdy commented Nov 30, 2020

@preslavmihaylov The tests are still failing.

Yeah, I've mentioned that in the description - tests whichare run with pull_request_target run with the workflow as-is from the master branch. The workflow from there doesn't have the new secret I've added.

After the diff is merged, the test will start passing, as is seen by the push variant of the step.

Sorry. I missed the comment. :))

Copy link
Collaborator

@mehdy mehdy left a comment

Choose a reason for hiding this comment

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

Everything looks good to me.

@preslavmihaylov preslavmihaylov merged commit d68ef8d into master Nov 30, 2020
@preslavmihaylov preslavmihaylov deleted the add-gitlab-ci-tests branch November 30, 2020 05:50
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 workflow changes available on repo branches, but not on forks
2 participants