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: Allow to edit commit message #185

Closed
wants to merge 6 commits into from

Conversation

rocktimsaikia
Copy link
Contributor

@rocktimsaikia rocktimsaikia commented Apr 2, 2023

Allows users to edit the selected commit message before committing

closes #110
cc @privatenumber


message = updatedMessage as string;
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of putting this behind an --edit flag, let's refactor the confirmation prompt to be:

  1. Commit
  2. Edit message

Copy link
Contributor Author

@rocktimsaikia rocktimsaikia Apr 3, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean by replacing the "Yes and No" confirmation prompt?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm okey.Makes sense.

@rocktimsaikia rocktimsaikia changed the title feat: Add --edit flag feat: Allow to edit commit message Apr 3, 2023
@rocktimsaikia
Copy link
Contributor Author

@privatenumber The change is done. Btw you have to update the demo terminal image in the readme.

@privatenumber
Copy link
Collaborator

Thanks, will try to review soon.

Would you be able to update the image in the README in a separate PR?

@rocktimsaikia
Copy link
Contributor Author

Sure. I can do that.

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.

Feature: Allow user to modify the commit message themselves on terminal.
2 participants