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

Use a file in the repo listing all the commits/timestamps for each result file #317

Open
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

lyricnz
Copy link
Contributor

@lyricnz lyricnz commented Nov 21, 2023

Per #309

  • file is manually generated (for now) runs as part of normal batch
  • no caching
  • works OK

@lyricnz
Copy link
Contributor Author

lyricnz commented Nov 21, 2023

results/git-commits.json is 3.3MB

looks like

{
    "NSW/baldersleigh": [
        "1700083023 aec31c8",
        "1698558578 2a9a881",
        "1696872777 aba8955",
        "1695160529 f3716b2",
        "1693407446 a91596f",
        "1691639736 3d67900",
        "1688432262 dfa48f6",
        "1688231116 379c988"
    ],
    "NSW/balmoral": [
        "1700083023 aec31c8",
        "1698562769 29dff7e",
        "1696877046 289d649",
        "1695160529 f3716b2",
        "1693407446 a91596f",
...

@lyricnz lyricnz marked this pull request as draft November 21, 2023 01:09
@lyricnz lyricnz marked this pull request as ready for review November 25, 2023 22:51
@lyricnz
Copy link
Contributor Author

lyricnz commented Nov 25, 2023

It has no caching, but that's no worse than current (except file is bigger), and maybe @LukePrior will add that to his service-thread stuff. Way beyond me.

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.

1 participant