Skip to content

Conversation

@tesseractjh
Copy link

Add use cache directives(including use cache: remote, use cache: private) for Next.js 16

@tesseractjh tesseractjh requested a review from a team as a code owner November 29, 2025 16:45
@github-actions github-actions bot added the JS label Nov 29, 2025
@tesseractjh tesseractjh requested a review from a team as a code owner November 29, 2025 17:22
Comment on lines 5 to 6
"editor.suggest.matchOnWordStartOnly": false,
"editor.formatOnSave": false
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you committed this by mistake.

Copy link
Author

Choose a reason for hiding this comment

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

oh i missed it. thanks

@tesseractjh tesseractjh force-pushed the js/use-cache-directives branch from 5b7c2d0 to c4f6606 Compare November 30, 2025 04:54
Copy link
Contributor

@asgerf asgerf left a comment

Choose a reason for hiding this comment

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

Thanks for the PR!

* ```
*/
class UseCacheDirective extends KnownDirective {
UseCacheDirective() { this.getDirectiveText().regexpMatch("use cache(: (remote|private))?") }
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
UseCacheDirective() { this.getDirectiveText().regexpMatch("use cache(: (remote|private))?") }
UseCacheDirective() { this.getDirectiveText().regexpMatch("use cache(:.*)?") }

It seems better to match anything here so we don't have to come back and update it if more cache modes are added in the future.

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