Skip to content

feat: add clear cache#2

Merged
inaie-makerx merged 3 commits intomainfrom
add-clear-cache
May 28, 2024
Merged

feat: add clear cache#2
inaie-makerx merged 3 commits intomainfrom
add-clear-cache

Conversation

@inaie-makerx
Copy link
Copy Markdown
Contributor

Add function to clear cache

Pull-Request Checklist

  • Code is up-to-date with the main branch
  • npm run lint passes with this change
  • npm test passes with this change
  • npm commitlint passes with this change
  • This pull request links relevant issues as Fixes #0000 "N/A"
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change

Please note that this repository uses conventional commits in combination with semantic-release to automate package publication. Therefore, your commit messages are critical, and the build process will lint them. If the build fails at the commitlint stage, please refer to this handy guide on how to update the commit messages

Comment thread src/cache.ts Outdated
Copy link
Copy Markdown
Contributor

@cuzzlor cuzzlor left a comment

Choose a reason for hiding this comment

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

Looks good apart from the one comment typo (see suggestion)

@robdmoore
Copy link
Copy Markdown
Member

Really nice work

@inaie-makerx inaie-makerx merged commit fda2093 into main May 28, 2024
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.

3 participants