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

Link to user's profile in a generated changelog #37

Merged
merged 1 commit into from
Mar 25, 2020

Conversation

MaxDesiatov
Copy link
Contributor

In a generated changelog it would be great if usernames also linked to corresponding GitHub user profiles. The tests are modified accordingly.

Copy link
Contributor

@AvdLee AvdLee left a comment

Choose a reason for hiding this comment

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

Hey Max!

Thanks for your contribution, this is great.
For my understanding, this is mostly for Changelog.md files, right?

GitHub is auto-linking from GitHub releases as you can see here.

Just want to know your thoughts behind this change 🙂

Thanks!

@ghost
Copy link

ghost commented Mar 24, 2020

Messages
📖

View more details on Bitrise

📖 GitBuddy-Package: Executed 44 tests, with 0 failures (0 unexpected) in 1.683 (1.732) seconds

GitBuddyCore.framework: Coverage: 89.38

File Coverage
ChangelogItem.swift 92.31%

Generated by 🚫 Danger Swift against 06e02bc

@MaxDesiatov
Copy link
Contributor Author

Hey Antoine! Yes, this change is specifically for changelogs posted outside of GitHub releases, where auto-linking doesn't work, Changelog.md being one of those. Hope this makes sense.

@AvdLee
Copy link
Contributor

AvdLee commented Mar 25, 2020

It does, makes total sense! Merging in 🙂

@AvdLee AvdLee merged commit 68abc6c into WeTransfer:master Mar 25, 2020
@ghost
Copy link

ghost commented Mar 25, 2020

Congratulations! 🎉 This was released as part of Release 2.3.0 🚀

@MaxDesiatov MaxDesiatov deleted the username-github-link branch March 31, 2020 11:10
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