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

feat: create last-modified mapping #343

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

line-o
Copy link
Contributor

@line-o line-o commented Jan 17, 2025

Whenever this package is built in CI a module
modules/frus.xqm is added to the built XAR.

This can then be used to optimize caching in hsg-shell.

Whenever this package is built in CI a module
`modules/frus.xqm` is added to the built XAR.

This can then be used to optimize caching in hsg-shell.
@line-o line-o force-pushed the feat/cacheability branch from 68136c0 to 0057038 Compare January 18, 2025 19:15
line-o added a commit to line-o/hsg-shell that referenced this pull request Jan 20, 2025
Utilizing the frus.xqm that is added with HistoryAtState/frus#343

The date in the map is not the last modified date in the database which might change frequently but the last commit date.
line-o added a commit to line-o/hsg-shell that referenced this pull request Jan 20, 2025
Utilizing the frus.xqm that is added with HistoryAtState/frus#343

The date in the map is not the last modified date in the database which might change frequently but the last commit date.
@line-o
Copy link
Contributor Author

line-o commented Jan 20, 2025

@joewiz pointed out that the script does only return the same date for all files. This is likely due to us only checking out a single commit in CI.
How should we proceed?

  • create separate workflow that checks out several commits so that the script compiles the correct mapping
  • run that script only on pushes to master - before releases, as the checkout may take a while

@line-o
Copy link
Contributor Author

line-o commented Jan 21, 2025

@joewiz and I found a different way to obtain the last committed date

$file='volumes%2Ffrus1981-88v01.xml'; \
curl -s "https://api.github.com/repos/HistoryAtState/frus/commits?path=$file&page=1&per_page=1" | \
jq ".[0] | .commit.author.date"

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