Include name of repo in cleanup log entries#1293
Include name of repo in cleanup log entries#1293Torbjorn-Svensson wants to merge 1 commit intogithub:mainfrom
Conversation
Contributed by STMicroelectronics Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
jeffwidman
left a comment
jeffwidman
left a comment
There was a problem hiding this comment.
Thanks for swinging by.
I'm curious--what is your use case where this would be helpful?
|
It would have been very helpful when debugging the issue that I reported in dependabot/dependabot-core#10493. @jeffwidman, if you have any idea what can be done about that issue, I would be happy to help out with both coding it and validating the change if needed. It's really frustrating to spend 15 min on each dependabot check to just pull the same image over and over again (yes I could work around it by tagging it manually, but that's only going to work for a short time and then it's out of date again). |
|
Looking at this again, sorry it's been so long. When you say "for repo", is that a git repo? Or some other type of package repo? Can you give me an example of what you're expecting this to log? Because if it's git repos, it should just be our standard repos unless you're running a fork... and in that case I'd expect a user to be aware that they're running a fork... so I'm still a little puzzled how this helped you in debugging. But I also haven't looked deeply at this code, so I'm probably misunderstanding something. |
|
My memory is also flushed on this topic, but from what I do remember, the With current implementation, the following can be seen in the log: With my change, I expect that the last 2 lines would have change to something like: (I have not checked what the hashes belongs to, so just take the above snippet as an example output and not literal.) I hope this makes more sense to you now. |
2 participants
Contributed by STMicroelectronics