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.

Sign up for GitHub

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

Jump to bottom

fixes #1921 #1922

Merged
merged 1 commit into from
Nov 20, 2019
Merged

fixes #1921 #1922

merged 1 commit into from
Nov 20, 2019

Conversation

dittodhole
Copy link

@dittodhole dittodhole commented Nov 19, 2019 •
edited
Loading

see #1921

@dittodhole dittodhole mentioned this pull request Nov 19, 2019
Closed
@asbjornu
Copy link
Member

Thanks for this, @dittodhole! Is the build failing due to the long branch name, @arturcic?

@arturcic
Copy link
Member

I noticed for some time that the appveyor only has this issue with long names. As far as I remember the chocolatey task fails on appveyor, but it should not run at all for PRs. Need to investigate. But you can merge if Azure Pipelines succeedes

@arturcic
Copy link
Member

Thank you so much for your contributions, @dittodhole 👍

@arturcic arturcic closed this Nov 20, 2019
@arturcic arturcic reopened this Nov 20, 2019
@arturcic arturcic merged commit a2be4a1 into GitTools:master Nov 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dittodhole @asbjornu @arturcic