-
Notifications
You must be signed in to change notification settings - Fork 31
chore: remove unnecessary lint-staged config and dependency #74
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
base: develop
Are you sure you want to change the base?
Conversation
|
cc: @drtechie |
WalkthroughThis change removes the lint-staged integration from the project. Specifically, it deletes the Changes
Possibly related issues
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
🔇 Additional comments (2)
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|



📋 Description
fixes PSMRI/AMRIT#98
JIRA ID: issue98
This PR removes the unused lint-staged configuration and dependency from the project. The motivation for this change is that the lint-staged configuration targeted .ts, .css, and .html files in the src/ directory, but none of these file types exist in the project. Additionally, the Husky pre-commit hook that invoked lint-staged was deleted to prevent commit failures after the dependency was removed. No other configuration files (such as .factorypath) were affected, as they are unrelated to this tooling.
✅ Type of Change
[ ] 🐞 Bug fix
[ ] ✨ New feature
[ ] 🔥 Breaking change
[ ] 🛠 Refactor
[x] ⚙️ Config change (configuration file or build script updates)
[ ] 📚 Documentation
[ ] 🧪 Tests
[ ] 🎨 UI/UX
[ ] 🚀 Performance
[x] 🧹 Chore (miscellaneous changes that don't modify src or test files)
ℹ️ Additional Information
Removed the lint-staged section from package.json
Uninstalled the lint-staged npm package and removed it from devDependencies
Deleted the .husky/pre-commit hook, which was causing commit failures after lint-staged was removed

Confirmed that .factorypath and other project files were unaffected
Changes were tested by running git commit to ensure no pre-commit errors occur and by verifying that the project installs and runs as expected
No functional code was changed; these updates are purely configuration and maintenance related.
Summary by CodeRabbit