-
Sub-task
-
Resolution: Fixed
-
Low
-
None
While i've mentioned that we can't easily link to source without setting up our own gitweb type solution which keeps the commits forever, I forgot about a nice solution we could implement before then with common git hosting sites..
Worked example with this job:
1. We know the FETCH_HEAD of the branch we are fetching (in that case its eed599a)
2. Github provide a way to access source by link: https://github.com/FMCorz/moodle/commit/eed599a/
3. We know the file and line number, so can link it directly to it.
4. It doesn't depend on branch name, which would become outdated
Problems:
- It will only work while github et al don't git gc the commits, they do do it, so these links won't work forever, but they are better than nothing and it seems it doesn't happen that frequently.
Only supports github, I agree we should make these tools git platform agnostic but is just an additional thing and will benefit 99% of our users and we can build it to support any known git repo hosting(Made it support github, bitbucket, gitorious in the end - we can add sites easily if they support web links to provide it)Depends onMDLSITE-3423so we don't rely on hash of smurf file - no worries, that'll be resolved soon- Merging problems might cause line number differences - hopefully not if people rebase - but its possible some auto-merges might slightly upset this
Cool!
- has a non-specific relationship to
-
MDLSITE-3644 Add execution context details to smurf reports
-
- Open
-
- has been marked as being related by
-
MDLSITE-4162 Make cibot report an explicit github/bitbucket diff url link
-
- Open
-
- is blocked by
-
MDLSITE-3423 Improve reporting of pre-checker results in the Tracker
-
- Closed
-
- will help resolve
-
MDLSITE-2803 Link smurf output to the source
-
- Closed
-