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

Replace commit author name with login #175

Merged

Conversation

mariotacke
Copy link
Contributor

GitHub markdown converts @-references with links to user profiles. Using
the full author name means potentially including a space which breaks this
linking behavior. Instead of the author name (Ex: Mario Tacke), this fix
uses the author login (Ex: mariotacke) to form a proper link in the
release notes and changelog).

This fixes #158

GitHub markdown converts @-references with links to user profiles. Using
the full author name means potentially including a space which breaks this
linking behavior. Instead of the author name (Ex: Mario Tacke), this fix
uses the author login (Ex: mariotacke) to form a proper link in the
release notes and changelog).

This fixes github-tools#158
@mariotacke
Copy link
Contributor Author

The build completed fine on Node 7.9 and 8. The Node 6 runner timed out on TravisCI. Please re-run to get a passing build.

@codecov-io
Copy link

Codecov Report

Merging #175 into master will decrease coverage by 46.08%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #175       +/-   ##
===========================================
- Coverage   84.34%   38.26%   -46.09%     
===========================================
  Files           7        7               
  Lines         345      345               
===========================================
- Hits          291      132      -159     
- Misses         54      213      +159
Impacted Files Coverage Δ
lib/src/Gren.js 5.28% <0%> (-75.97%) ⬇️
lib/src/GitHubInfo.js 80.95% <0%> (-4.77%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c923505...7328564. Read the comment docs.

@alexcanessa alexcanessa merged commit beff4c9 into github-tools:master Jun 18, 2018
@mariotacke mariotacke deleted the fix-158-commit-author-name branch June 18, 2018 15:18
@alexcanessa
Copy link
Member

@all-contributors please add @mariotacke for code

@allcontributors
Copy link
Contributor

@alexcanessa

I've put up a pull request to add @mariotacke! 🎉

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.

Commit author's name link is incorrect
3 participants