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

Enable fetching retried GitLab jobs #87

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mvgijssel
Copy link
Contributor

fixes #86

@sonarcloud
Copy link

sonarcloud bot commented Sep 15, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ericboucher
Copy link

Thanks @mvgijssel. Looks like a reasonable addition. If you're ok with it, I'd prefer to merge this to the main branch directly. Let me know if that works for you,

@mvgijssel
Copy link
Contributor Author

Thanks @mvgijssel. Looks like a reasonable addition. If you're ok with it, I'd prefer to merge this to the main branch directly. Let me know if that works for you,

Sure no problem!

@ericboucher ericboucher changed the base branch from legacy-stable to main October 29, 2022 20:57
@ericboucher
Copy link

Thanks! Do you mind rebasing/restarting from main?. Let me know if you need a hand :)

@tayloramurphy
Copy link

@ericboucher is this PR still relevant?

@glauberferreiranees
Copy link

@ericboucher is this PR still relevant?

Yes. Extremely relevant for me. Is there a plan to release a new version with this PR?

@edgarrmondragon
Copy link
Member

@ericboucher is this PR still relevant?

Yes. Extremely relevant for me. Is there a plan to release a new version with this PR?

@glauberferreiranees I can review a new PR that fixes the merge conflicts if you're interested :)

@glauberferreiranees
Copy link

@ericboucher is this PR still relevant?

Yes. Extremely relevant for me. Is there a plan to release a new version with this PR?

@glauberferreiranees I can review a new PR that fixes the merge conflicts if you're interested :)

I would be extremely grateful @edgarrmondragon

@edgarrmondragon
Copy link
Member

@glauberferreiranees Great! I'll be on the lookout for your PR.

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.

(Optionally) include retried GitLab jobs
5 participants