Skip to content
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

feat(src): clean up progress bar #28

Merged
merged 1 commit into from
Jan 27, 2025
Merged

feat(src): clean up progress bar #28

merged 1 commit into from
Jan 27, 2025

Conversation

chiefmikey
Copy link
Owner

This pull request includes changes to the progress bar format and its usage in the dependency analysis tool. The most important changes include updating the progress bar format string, modifying the progress bar initialization and update logic, and ensuring the final update correctly reflects the completion status.

Changes to progress bar format and usage:

  • src/constants.ts: Updated the PROGRESS_FORMAT string in CLI_STRINGS to a new format that includes currentDeps, totalDeps, and dep instead of currentFiles, totalFiles, and percentage.
  • src/index.ts: Modified the progress bar initialization to use the updated PROGRESS_FORMAT and added new options such as hideCursor, clearOnComplete, forceRedraw, and linewrap.
  • src/index.ts: Updated the progress bar during the analysis process to use currentDeps, totalDeps, and dep instead of currentDep.
  • src/index.ts: Ensured the final update to the progress bar shows 100% completion with the correct values for currentDeps, totalDeps, and dep.

@chiefmikey chiefmikey merged commit 164df24 into main Jan 27, 2025
2 checks passed
@chiefmikey chiefmikey deleted the clean-up branch January 27, 2025 23:44
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.

1 participant