Skip to content

perf(listArchives): add cache for posts#5624

Merged
SukkaW merged 2 commits intohexojs:masterfrom
D-Sketon:perf/list-archives
Feb 13, 2025
Merged

perf(listArchives): add cache for posts#5624
SukkaW merged 2 commits intohexojs:masterfrom
D-Sketon:perf/list-archives

Conversation

@D-Sketon
Copy link
Member

@D-Sketon D-Sketon commented Jan 25, 2025

What does it do?

add cache for posts when relative_link is true(we cannot use cache in template)
before
{5BA484CC-9C9D-493D-ADE9-05545841F6A0}
after
{470B797A-99C5-41A5-996E-B67548D22020}

Screenshots

Pull request tasks

  • Add test cases for the changes.
  • Passed the CI test.

@github-actions
Copy link

How to test

git clone -b perf/list-archives https://github.com/D-Sketon/hexo.git
cd hexo
npm install
npm test

@coveralls
Copy link

Pull Request Test Coverage Report for Build 12961774252

Details

  • 7 of 7 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 99.492%

Totals Coverage Status
Change from base Build 12902994421: 0.0%
Covered Lines: 9593
Relevant Lines: 9642

💛 - Coveralls

Copy link
Member

@SukkaW SukkaW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am wondering if it is possible to add caches directly to the warehouse though, and invalidate automatically when the data is updated.

@SukkaW SukkaW merged commit bc07e1a into hexojs:master Feb 13, 2025
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants