Copilot commented on code in PR #409:
URL: https://github.com/apache/logging-parent/pull/409#discussion_r2137369179
##########
.github/workflows/deploy-site-reusable.yaml:
##########
@@ -105,11 +103,13 @@ jobs:
git config user.name "ASF Logging Services RM"
git config user.email priv...@logging.apache.org
- # Checking out a new branch will delete the `node_modules` folder
- - name: Save Node.js modules
- shell: bash
- run: |
- zip -q0X "$RUNNER_TEMP"/node_modules.zip node_modules
+ # Checking out a new branch will delete the `node_modules` folder,
+ # so we need to save the cache here.
+ - name: Save Node.js cache
+ uses: actions/cache/restore@5a3ec84eff668545956fd18022155c47e93e2684
# 4.2.3
Review Comment:
The 'Save Node.js cache' step is using the 'actions/cache/restore' action,
which is intended for cache restoration. Consider switching to the appropriate
caching action or configuration that supports saving the cache.
```suggestion
uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 #
4.2.3
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@logging.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
