Skip to content

Conversation

@rajibhasenraju
Copy link
Contributor

@rajibhasenraju rajibhasenraju commented Jan 4, 2026

Description

Fixed a typo in the filename encrypt-sectrets.phpencrypt-secrets.php

Changes

  • Renamed encrypt-sectrets.php to encrypt-secrets.php
  • Updated reference in package.json prearchive script

Type of Change

  • Bug fix (typo correction)
  • Documentation/filename correction

Checklist

  • Both file rename and reference update are included
  • No functionality changes, only typo correction

Summary by CodeRabbit

  • Bug Fixes

    • Fixed typos in build/archive scripts so packaging and pre-archive commands run correctly during releases.
  • Style

    • Added a trailing newline and minor formatting cleanup for file cleanliness.

✏️ Tip: You can customize this high-level summary in your review settings.

fix: update package.json to reference correct filename
Copilot AI review requested due to automatic review settings January 4, 2026 13:12
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 4, 2026

📝 Walkthrough

Walkthrough

Corrected a misspelled script filename in package.json and composer.json from "encrypt-sectrets.php" to "encrypt-secrets.php", and added a trailing newline to encrypt-secrets.php for EOF formatting consistency.

Changes

Cohort / File(s) Change Summary
Script typo fixes
package.json, composer.json
Corrected misspelling encrypt-sectrets.phpencrypt-secrets.php in prearchive script (package.json) and archive exclude list (composer.json).
EOF formatting
encrypt-secrets.php
Added a trailing newline at end of file for formatting/EOF cleanliness.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped through code with careful feet,
Found a secret spelling incomplete.
I nudged a newline, fixed the name,
Now scripts and secrets play the same. ✨

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: correcting a typo in the encrypt-secrets.php filename across multiple files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ced69ab and 6e34864.

📒 Files selected for processing (1)
  • composer.json
🔇 Additional comments (1)
composer.json (1)

145-145: LGTM! Typo corrected in archive exclude list.

The filename has been correctly updated from "encrypt-sectrets.php" to "encrypt-secrets.php", fixing the misspelling in the archive exclusion configuration. No remaining references to the old misspelled filename exist in the codebase.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

Copilot AI left a 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.php to encrypt-secrets.php
  • Updated the prearchive script in package.json to 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",
Copy link

Copilot AI Jan 4, 2026

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.

Copilot uses AI. Check for mistakes.
Copy link
Contributor

@coderabbitai coderabbitai bot left a 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

📥 Commits

Reviewing files that changed from the base of the PR and between a355a9f and ced69ab.

📒 Files selected for processing (2)
  • encrypt-secrets.php
  • package.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.

@superdav42 superdav42 merged commit 2da607b into Multisite-Ultimate:main Jan 6, 2026
10 of 12 checks passed
@superdav42
Copy link
Collaborator

@rajibhasenraju Thanks for the PR. It was spot on.

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.

2 participants