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

🚀 UPDATE: v0.2.7 #21

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

🚀 UPDATE: v0.2.7 #21

wants to merge 7 commits into from

Conversation

warengonzaga
Copy link
Owner

This pull request refactors the App component and introduces a new Logo component to simplify and centralize the display of the application logo and version information. The changes primarily focus on improving the code structure and readability.

Refactoring and code structure improvements:

  • source/app.js: Removed imports for Text, Newline, BigText, and Gradient, and replaced the inline logo and version information display with the new Logo component. [1] [2]
  • source/utils/commit.js: Added the Logo component to wrap the commit message prompt and removed unnecessary whitespace. [1] [2] [3]

New component introduction:

  • source/utils/logo.js: Created a new Logo component to encapsulate the display of the application logo, version, author, and help information.

Minor formatting adjustments:

  • source/app.js: Adjusted the formatting of console log messages for better readability.

@warengonzaga warengonzaga added chore Project Housekeeping (Issue/PR) fix Bug Fix (PR) hacktoberfest-accepted Hacktoberfest (PR) maintainers only Assigned to Project Maintainers (Issue/PR) tweak Enhancements (Issue/PR) labels Oct 15, 2024
@warengonzaga warengonzaga self-assigned this Oct 15, 2024
@warengonzaga
Copy link
Owner Author

I think good addition to this PR is adding the logo to flags such as --version and --help what do you think? @spc-28

@warengonzaga
Copy link
Owner Author

I created a new issue here, #22. Maybe one of you @spc-28 @Abhinav-Bansal751 can take it.

@warengonzaga warengonzaga added the release Release Status (PR) label Oct 15, 2024
@spc-28
Copy link
Contributor

spc-28 commented Oct 15, 2024

@warengonzaga its a good idea.
Can i work on it?

@warengonzaga
Copy link
Owner Author

@warengonzaga its a good idea. Can i work on it?

Yes, please!

@warengonzaga warengonzaga linked an issue Oct 19, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Project Housekeeping (Issue/PR) fix Bug Fix (PR) hacktoberfest-accepted Hacktoberfest (PR) maintainers only Assigned to Project Maintainers (Issue/PR) release Release Status (PR) tweak Enhancements (Issue/PR)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

keep the logo at the top of the CLI
2 participants