-
Notifications
You must be signed in to change notification settings - Fork 106
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 commit of latest version if known #874
Link to commit of latest version if known #874
Conversation
d249a23
to
de07fe5
Compare
Codecov Report
@@ Coverage Diff @@
## master #874 +/- ##
==========================================
+ Coverage 97.03% 97.07% +0.03%
==========================================
Files 61 61
Lines 3577 3587 +10
Branches 512 512
==========================================
+ Hits 3471 3482 +11
Misses 64 64
+ Partials 42 41 -1
Continue to review full report at Codecov.
|
3499676
to
354c14f
Compare
cf1b4d5
to
cd55786
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two small changes:
- Squash commits
- Add news file
575b60d
to
29baad0
Compare
Hmm, there are two failures flagged by Travis CI:
I'll force push again to see if the #1008.6 issue improves, there's nothing I can do about the latter. |
- Add ProjectVersion.commit_url This is to let us store URLs pointing to tagged commits. - GitHub: Store received commit URLs for versions - Set commit_url on version objects - Add Project.latest_version_object property Fixes: fedora-infra#590 Signed-off-by: Nils Philippsen <[email protected]>
29baad0
to
1ec5e62
Compare
@nphilipp 3.8 could be safely ignored |
This only links the commit from the latest version field, where we can make it explicit that it's a commit, not a release page or anything: