Skip to content

📦 new: add context to git commit message#14

Merged
warengonzaga merged 12 commits intowarengonzaga:devfrom
Abhinav-Bansal751:dev
Oct 11, 2024
Merged

📦 new: add context to git commit message#14
warengonzaga merged 12 commits intowarengonzaga:devfrom
Abhinav-Bansal751:dev

Conversation

@Abhinav-Bansal751
Copy link
Copy Markdown
Contributor

Pull Request

Summary

This PR updates the config.json file to improve the commit message generation logic. The changes include the addition of a new prompt format for better categorization and context in commit messages.

Changes Made

  • Updated the emoji prompt to include a structured format: <emoji> category (context): title..

Related Issues

@Abhinav-Bansal751
Copy link
Copy Markdown
Contributor Author

@warengonzaga can u please check if its correct.

@warengonzaga warengonzaga self-requested a review October 6, 2024 10:25
Copy link
Copy Markdown
Owner

@warengonzaga warengonzaga left a comment

Choose a reason for hiding this comment

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

Please check my requested changes. Looks like it needs a complete git message convention. Not sure what to use in (context) or (scope) I think for now let's use file name of staged file. I leave a reference file where you can edit it.

@warengonzaga
Copy link
Copy Markdown
Owner

The git commit convention here is my personal convention when building open source projects. I think I have to make a complete specs of my preference.

@warengonzaga warengonzaga self-requested a review October 6, 2024 13:55
Copy link
Copy Markdown
Owner

@warengonzaga warengonzaga left a comment

Choose a reason for hiding this comment

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

Hey there, thanks for the contribution. Could you take a look at my requested changes?

@warengonzaga
Copy link
Copy Markdown
Owner

Kindly fix the conflicting file issue @Abhinav-Bansal751. The best way to do this is to merge the changes from the dev branch and then use a web editor to merge the changes in the dev branch to your copy.

@warengonzaga warengonzaga added hacktoberfest-accepted Hacktoberfest (PR) tweak Enhancements (Issue/PR) feature Feature Request (Issue) labels Oct 6, 2024
@warengonzaga warengonzaga changed the title update config.json 📦 new: add context to git commit message Oct 6, 2024
@warengonzaga warengonzaga self-requested a review October 7, 2024 04:37
Copy link
Copy Markdown
Owner

@warengonzaga warengonzaga left a comment

Choose a reason for hiding this comment

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

Kindly check the comments and click "resolve conversation" if you already updated the code. It should show "outdated" if you made changes. Lastly, I think the best way is to delete the yarn.lock file I can handle the rest.

@warengonzaga
Copy link
Copy Markdown
Owner

I fixed the conflict @Abhinav-Bansal751 kindly resolve the requested changes thank you!

@Abhinav-Bansal751
Copy link
Copy Markdown
Contributor Author

Sir,Should i make another commit ?i have deleted "magicc": "^0.2.1", from package.json, made config.json same as earlier.
Also i will commit with yarn.lock file deleted this time as u have mentioned

@warengonzaga
Copy link
Copy Markdown
Owner

Yes @Abhinav-Bansal751 please made the requested changes.

Copy link
Copy Markdown
Owner

@warengonzaga warengonzaga left a comment

Choose a reason for hiding this comment

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

I have another requested changes @Abhinav-Bansal751 kindly take a look.

Copy link
Copy Markdown
Owner

@warengonzaga warengonzaga left a comment

Choose a reason for hiding this comment

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

All good @Abhinav-Bansal751 however, I have some requested changes.

Copy link
Copy Markdown
Owner

@warengonzaga warengonzaga left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the contribution!

@warengonzaga warengonzaga merged commit b6ae7ef into warengonzaga:dev Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Feature Request (Issue) hacktoberfest-accepted Hacktoberfest (PR) tweak Enhancements (Issue/PR)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants