Skip to content

Replace AppVeyor badge with Azure Pipelines #445

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
Merged
merged 2 commits into from
Jun 15, 2019
Merged

Replace AppVeyor badge with Azure Pipelines #445

merged 2 commits into from
Jun 15, 2019

Conversation

AArnott
Copy link
Contributor

@AArnott AArnott commented May 29, 2019

Closes #416

@MarcoRossignoli
Copy link
Collaborator

Mmm...I didn't found that but only https://docs.microsoft.com/en-us/rest/api/azure/devops/build/badge/get%20build%20badge%20data?view=azure-devops-rest-5.0
Thank's

MarcoRossignoli
MarcoRossignoli approved these changes May 29, 2019
View reviewed changes
Copy link
Collaborator

@MarcoRossignoli MarcoRossignoli left a comment

Choose a reason for hiding this comment

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

LGTM

@MarcoRossignoli MarcoRossignoli requested a review from tonerdo May 29, 2019 07:05
@AArnott
Copy link
Contributor Author

AArnott commented May 29, 2019

FYI: @MarcoRossignoli You can get the status badge markdown from here:
image

@tonerdo
Copy link
Collaborator

tonerdo commented May 29, 2019

Will merge after #422

@tonerdo
Copy link
Collaborator

tonerdo commented Jun 4, 2019

@AArnott qq: anyway to override using git height for the patch version? I wanna make a new release and it doesn't really have the public release incremented by just one. E.g. 2.6.1 -> 2.6.2

@AArnott
Copy link
Contributor Author

AArnott commented Jun 4, 2019

@tonerdo Your question looks a bit off topic for this PR. This PR just replaces the README badge to reflect that you build on Azure Pipelines now. Wanna open another issue to discuss?

@tonerdo
Copy link
Collaborator

tonerdo commented Jun 5, 2019

@AArnott yeah it was just a random question. I thought to ask here since this was an open PR. Nevermind that now, I've figured out what I need to do

@MarcoRossignoli
Copy link
Collaborator

@AArnott can you resolve so we can merge?

@AArnott
Copy link
Contributor Author

AArnott commented Jun 12, 2019

done

tonerdo
tonerdo approved these changes Jun 15, 2019
View reviewed changes
Copy link
Collaborator

@tonerdo tonerdo left a comment

Choose a reason for hiding this comment

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

Thanks @AArnott

@tonerdo tonerdo merged commit cc60b0d into coverlet-coverage:master Jun 15, 2019
@AArnott AArnott deleted the fix416 branch June 15, 2019 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tonerdo tonerdo tonerdo approved these changes

@MarcoRossignoli MarcoRossignoli MarcoRossignoli approved these changes

Assignees
No one assigned
Labels
documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update build badge AZP
3 participants
@AArnott @MarcoRossignoli @tonerdo