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 support for azure devops and visualstudio.com #72

Merged
merged 8 commits into from
Nov 8, 2018

Conversation

jordhan94
Copy link
Contributor

This solves issue #67. auto-changelog is a great tool and I want to use it on some projects but it doesn't support Azure DevOps.

Let me know if I must make some changes, I'll do it happily ❤️.

Copy link
Owner

@cookpete cookpete left a comment

Choose a reason for hiding this comment

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

Nice work so far!

src/releases.js Outdated Show resolved Hide resolved
CHANGELOG.md Outdated
@@ -4,287 +4,276 @@ All notable changes to this project will be documented in this file. Dates are d

Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).

#### [v1.8.1](https://github.com/CookPete/auto-changelog/compare/v1.8.0...v1.8.1)
#### [v1.8.1](https://github.com/Hwigaro/auto-changelog/compare/v1.8.1...v1.8.1)
Copy link
Owner

Choose a reason for hiding this comment

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

You don't need to include any changes to the CHANGELOG.md in PRs. This is done automatically when creating a new version.

src/remote.js Outdated Show resolved Hide resolved
src/remote.js Outdated Show resolved Hide resolved
src/remote.js Outdated
}

return {
hostname,
url: `${protocol}//${hostname}/${repo}`
url: `${protocol}//${hostname}/${repo}`,
project: `${protocol}//${hostname}/${project}`
Copy link
Owner

Choose a reason for hiding this comment

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

I think project is a weird name for this property. It looks like it's only used as the base url for issues? In which case maybe it should be issueUrl or something.

Copy link
Contributor Author

@jordhan94 jordhan94 Nov 8, 2018

Choose a reason for hiding this comment

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

Yes, it is used only for the issues url. But, since this url is used by Azure/Visualstudio as a base url for many other project functionalities, I named it project.

Copy link
Contributor Author

@jordhan94 jordhan94 Nov 8, 2018

Choose a reason for hiding this comment

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

Maybe projectUrl would be a more descriptive name.

test/data/remotes.js Outdated Show resolved Hide resolved
@cookpete cookpete merged commit 1a48b23 into cookpete:master Nov 8, 2018
@cookpete
Copy link
Owner

cookpete commented Nov 8, 2018

Thanks @Hwigaro!

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.

2 participants