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

Proposal: same report file structure for one- and two-branch runs #393

Open
denny opened this issue Mar 27, 2021 · 0 comments
Open

Proposal: same report file structure for one- and two-branch runs #393

denny opened this issue Mar 27, 2021 · 0 comments

Comments

@denny
Copy link
Contributor

denny commented Mar 27, 2021

I was thinking about the HTML report file layout again a few days ago (I know, it's a rock n roll life I lead), and I was wondering if there's any reason for putting the files in a different place when running against a single branch vs when doing a compare between two branches, or if that difference is just an artefact of the order the features were originally implemented in.

If the files always went into a directory named after the branch inside a single top-level directory** then I think
(a) it would be possible to generate reports for every branch on your system if you want (in any order and combination of runs), with all the reports going into their own directories instead of overwriting each other, and
(b) once that's working, it should be fairly easy to have it add menu links at the top of each new report it generates for all the branches already on your disk, and
(c) It should also be possible*** to rewrite the menu section of the reports in every branch, every time a new branch is run - so that you can click from the older ones to the newer ones as well.

** Exactly like the compare directory works now, but with a more generic name because it would be used for every run type - branches maybe?
*** Carefully not saying 'easy' for this one; I haven't looked at how that HTML gets generated yet.

Anyway:

  1. Does this make sense?
  2. If yes to 1; do you want it? 🙂 Should I take a swing at it? The first bit, initially, although I actually fancy my chances more on the second bit, so if I get the first bit working then I'll probably keep going.
  3. If 'yes' to 2; would anyone be willing to volunteer as my phone email a friend if I get a bit stuck in the details on this one? Or should I just come and cry on a partial PR at that point? 😃 I think I figured out most of the HTML report file structure stuff for Fix compare file structure #378, but ... code is tricksy.

Oh, 4. Do I need to think about any of the other report formats, or are they independent enough of each other that I can do the change for HTML without disturbing the rest for now?

Aaaand 5. Ugh. I've just remembered that in #378 I tiptoed around some nasty code without disturbing it - I'm pretty sure I recall global path variables in the near vicinity of this change. Is 'before this' the right time to look at refactoring that to be less fragile/coupled/ugh?

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

No branches or pull requests

1 participant