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

Check travis pipeline #1592

Merged
merged 32 commits into from
Sep 7, 2020
Merged

Check travis pipeline #1592

merged 32 commits into from
Sep 7, 2020

Conversation

maxmarkus
Copy link
Contributor

@maxmarkus maxmarkus commented Aug 31, 2020

No tags were available locally due to the shallow cloning of travis.
This PR adds a git pull --tags if no local tag was found previously. Restricted this to 500, to not download the full repo. Added another check afterwards to verify a tag was found.

@JohannesDoberer JohannesDoberer added bug Something isn't working internal Stuff like NFRs, CI pipeline labels Sep 1, 2020
@JohannesDoberer JohannesDoberer added this to the Sprint 12 milestone Sep 1, 2020
@maxmarkus maxmarkus added the WIP Work in progress label Sep 1, 2020
@JohannesDoberer JohannesDoberer modified the milestones: Sprint 12, Sprint 13 Sep 2, 2020
@maxmarkus maxmarkus removed the WIP Work in progress label Sep 4, 2020
@JohannesDoberer JohannesDoberer self-assigned this Sep 4, 2020
@ndricimrr ndricimrr self-assigned this Sep 4, 2020
@maxmarkus maxmarkus merged commit e6a267d into SAP:master Sep 7, 2020
@maxmarkus maxmarkus deleted the check-travis-pipeline branch September 7, 2020 16:34
@ndricimrr ndricimrr mentioned this pull request Sep 9, 2020
stanleychh pushed a commit to stanleychh/luigi that referenced this pull request Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working internal Stuff like NFRs, CI pipeline
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants