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

✨ Support commit's URL and issues autolink in changelog #16

Merged
merged 6 commits into from
Sep 26, 2018

Conversation

bpetetot
Copy link
Collaborator

@bpetetot bpetetot commented Sep 25, 2018

Closes #15

gitmoji-changelog-core:

  • repoInfo.js: Extract git repo info from package.json, if not exists from .git remote origin
  • Add meta.repository object in the object generated by the core. it contains meta-information about the repository: { type, domain, user, project, url, bugsUrl }
  • Should support github, gitlab and bitbucket

gitmoji-changelog-markdown:

  • Add short hash with URL in changelog (7 chars)
  • Add autolink to issues and PRs in subject and body

Here the result: CHANGELOG.md

@bpetetot bpetetot changed the title ✨ Support commit url in changelog ✨ Support commit URL in changelog Sep 25, 2018
@bpetetot bpetetot changed the title ✨ Support commit URL in changelog ✨ Support commit's URL in changelog Sep 25, 2018
@bpetetot bpetetot changed the title ✨ Support commit's URL in changelog ✨ Support commit's URL and issues autolink in changelog Sep 26, 2018
@bpetetot bpetetot self-assigned this Sep 26, 2018
Copy link
Owner

@frinyvonnick frinyvonnick 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 👏

const packageInfo = await readPkgUp()
packageJson = packageInfo.pkg || {}
} catch (e) {
packageJson = {}
Copy link
Owner

Choose a reason for hiding this comment

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

packageJson is already empty, no ?

packageJson.repository = { url }
normalizePackageData(packageJson)
} catch (e) {
packageJson = {}
Copy link
Owner

Choose a reason for hiding this comment

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

here you can return null directly we don't want to go further ?

@bpetetot bpetetot merged commit 871f164 into master Sep 26, 2018
@bpetetot bpetetot deleted the feat/package-parser branch September 26, 2018 19:33
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