-
-
Notifications
You must be signed in to change notification settings - Fork 70
fix: correct typo in encrypt-secrets.php filename #317
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: correct typo in encrypt-secrets.php filename #317
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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", |
Copilot
AI
Jan 4, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.