-
-
Notifications
You must be signed in to change notification settings - Fork 2
Replace src/print/ folder with tree-dump package #12
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
Conversation
Co-authored-by: streamich <9773803+streamich@users.noreply.github.com>
Co-authored-by: streamich <9773803+streamich@users.noreply.github.com>
src/print/ folder
streamich
left a 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.
- Remove
package-lock.json. - Use
yarnto install dependencies. - Run
yarn testandyarn buildto validate everything builds.
Co-authored-by: streamich <9773803+streamich@users.noreply.github.com>
Done! Removed |
package.json
Outdated
| }, | ||
| "dependencies": {}, | ||
| "dependencies": { | ||
| "tree-dump": "^1.0.3" |
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.
Remove it from dependencies, as it is not used.
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.
Removed tree-dump from dependencies. Since the src/print/ folder was completely removed and no code actually uses the tree-dump package, the dependency was unnecessary. Commit 61ff8fb.
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.
Revert yarn.lock changes.
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.
Reverted yarn.lock to original state before the changes. Commit 61ff8fb.
Co-authored-by: streamich <9773803+streamich@users.noreply.github.com>
|
🎉 This PR is included in version 1.8.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This PR removes the
src/print/folder and replaces its functionality with thetree-dumpnpm package, which provides identical functionality.Changes Made
Removed entire
src/print/folder including:printTree.ts- function for printing general tree structuresprintBinary.ts- function for printing binary tree structurestypes.ts-Printableinterface definition__tests__/printTree.spec.ts,__tests__/printBinary.spec.ts)Added
tree-dump@^1.0.3as a dependencyWhy This Change?
The
tree-dumppackage provides the exact same functionality as the removedsrc/print/folder:printTreeandprintBinaryfunction signatures and behaviorPrintableinterfaceImpact
Usage
The functionality is now available through the
tree-dumppackage:Fixes #11.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.