Skip to content

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