fix: correct typo in encrypt-secrets.php filename #317
fix: correct typo in encrypt-secrets.php filename #317superdav42 merged 3 commits intoUltimate-Multisite:mainfrom
Conversation
Corrected typo in filename
fix: update package.json to reference correct filename
📝 WalkthroughWalkthroughCorrected a misspelled script filename in package.json and composer.json from "encrypt-sectrets.php" to "encrypt-secrets.php", and added a trailing newline to Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
Pre-merge checks✅ Passed checks (3 passed)
📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🔇 Additional comments (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This pull request fixes a typo in the filename encrypt-sectrets.php by renaming it to the correctly spelled encrypt-secrets.php. The PR updates the corresponding reference in the package.json prearchive script to use the corrected filename.
- Renamed file from
encrypt-sectrets.phptoencrypt-secrets.php - Updated the
prearchivescript inpackage.jsonto reference the corrected filename
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Updated the prearchive script to reference the correctly spelled filename |
| encrypt-secrets.php | Renamed file (diff shows no functional changes, only whitespace/line number adjustments) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "build:dev": "npm run copylibs && npm run uglify && npm run cleancss && npm run makepot", | ||
| "build:translate": "npm run copylibs && npm run uglify && npm run cleancss && npm run makepot && npm run translate", | ||
| "prearchive": "php encrypt-sectrets.php", | ||
| "prearchive": "php encrypt-secrets.php", |
There was a problem hiding this comment.
The reference in package.json has been correctly updated, but there's still another reference to the old filename "encrypt-sectrets.php" in composer.json line 145 (in the archive.exclude array) that also needs to be updated to "encrypt-secrets.php" for consistency.
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
encrypt-secrets.phppackage.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Agent
🔇 Additional comments (1)
encrypt-secrets.php (1)
31-31: LGTM! Trailing newline added for POSIX compliance.The addition of a trailing newline is good practice and aligns with standard file formatting conventions.
|
@rajibhasenraju Thanks for the PR. It was spot on. |
Description
Fixed a typo in the filename
encrypt-sectrets.php→encrypt-secrets.phpChanges
encrypt-sectrets.phptoencrypt-secrets.phppackage.jsonprearchive scriptType of Change
Checklist
Summary by CodeRabbit
Bug Fixes
Style
✏️ Tip: You can customize this high-level summary in your review settings.