Skip to content

refactor: replace deps:reset with clean:install and simplify clean sc…#16

Merged
sobeaa merged 2 commits intomainfrom
13-refactor-npm-scripts-for-cleaning-the-project
Apr 25, 2026
Merged

refactor: replace deps:reset with clean:install and simplify clean sc…#16
sobeaa merged 2 commits intomainfrom
13-refactor-npm-scripts-for-cleaning-the-project

Conversation

@sobeaa
Copy link
Copy Markdown
Member

@sobeaa sobeaa commented Apr 25, 2026

Summary

This pull request updates the project's clean and dependency reset scripts to improve clarity and maintain consistency. The main changes involve renaming and reorganizing npm scripts, as well as updating documentation to reflect these changes.

Script and documentation updates:

  • Replaced the deps:reset script with clean:install, and updated the documentation in CONTRIBUTING.md to reference the new script. (CONTRIBUTING.md, CONTRIBUTING.mdL36-R36)
  • Refactored npm scripts in package.json:
    • Changed the clean script to run both clean:install and clean:package.
    • Renamed predeps:reset to preclean:install and deps:reset to clean:install.
    • Added clean:package to remove dist and coverage directories. (package.json, package.jsonL35-R38)

Related issue

Closes #13

Type of change

  • Bug fix
  • Feature
  • Refactor
  • Documentation
  • Test improvement

Validation

  • npm run format:check
  • npm run lint
  • npm run typecheck
  • npm run knip
  • npm test
  • npm run build

Checklist

  • PR targets main.
  • The change is linked to an issue, or an issue is not required.
  • The solution is minimal and intentional.
  • Tests were added or updated for behavior changes.
  • Documentation was updated where relevant.

@sobeaa sobeaa linked an issue Apr 25, 2026 that may be closed by this pull request
@sobeaa sobeaa requested a review from Copilot April 25, 2026 09:58
Copy link
Copy Markdown

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 PR refactors the project’s npm “clean/reset dependencies” workflow by renaming deps:reset to clean:install, expanding clean to run both dependency reinstall and artifact cleanup, and updating contributor documentation accordingly.

Changes:

  • Replaced deps:reset/predeps:reset with clean:install/preclean:install.
  • Updated clean to run clean:install and clean:package, and added clean:package to remove dist and coverage.
  • Updated CONTRIBUTING.md to reference npm run clean:install.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
package.json Renames and reorganizes clean/reset scripts; adds clean:package and makes clean orchestrate both install + package cleanup.
CONTRIBUTING.md Updates contributor guidance to use npm run clean:install instead of deps:reset.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sobeaa sobeaa merged commit bd18335 into main Apr 25, 2026
7 checks passed
@sobeaa sobeaa deleted the 13-refactor-npm-scripts-for-cleaning-the-project branch April 25, 2026 10:05
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.

Refactor npm scripts for cleaning the project

2 participants