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

Feature Request: Summarize all commit #88

Open
Umiiii opened this issue Mar 13, 2023 · 5 comments
Open

Feature Request: Summarize all commit #88

Umiiii opened this issue Mar 13, 2023 · 5 comments
Labels
enhancement New feature or request

Comments

@Umiiii
Copy link

Umiiii commented Mar 13, 2023

Based on the multiple diff, can we summarise a sentence for creating Pull Request?

Also, I think we can add some configurations to ignore some files

@zurawiki
Copy link
Owner

zurawiki commented Mar 17, 2023

Based on the multiple diff, can we summarize a sentence for creating Pull Request?

How would you use this in your workflow today?

Also, I think we can add some configurations to ignore some files

We do ignore some files! The default files are at

static DEFAULT_FILES_TO_IGNORE: &[&str; 4] = &[
"package-lock.json",
"yarn.lock",
"pnpm-lock.yaml",
"Cargo.lock",
];

You can configure them per-repo with gptcommit config set file_ignore

@zurawiki zurawiki added the enhancement New feature or request label Mar 18, 2023
@zurawiki zurawiki changed the title Feature Request: Summarise all commit Feature Request: Summarize all commit Mar 18, 2023
@Umiiii
Copy link
Author

Umiiii commented Mar 20, 2023

Based on the multiple diff, can we summarize a sentence for creating Pull Request?

How would you use this in your workflow today?

Also, I think we can add some configurations to ignore some files

We do ignore some files! The default files are at

static DEFAULT_FILES_TO_IGNORE: &[&str; 4] = &[
"package-lock.json",
"yarn.lock",
"pnpm-lock.yaml",
"Cargo.lock",
];

You can configure them per-repo with gptcommit config set file_ignore

Sure!
Suppose you have made several commits in your local workspace, and now you want to submit them as a pull request to be merged into the master branch later. To make the process smoother, it's important to provide a clear and concise summary of the changes you've made. This summary should include the main features or fixes implemented by your commits, as well as any relevant information about the project or the development process. A well-written summary can help the reviewer understand your changes and make the merging process more efficient.

@zurawiki
Copy link
Owner

Hey @Umiiii , thanks for the feedback. Right now, making summaries across multiple commits is not support as gptcommit is for git commits

@Niek
Copy link

Niek commented Jul 9, 2023

This would also be a great feature to support changelog creation.

@carlin-q-scott
Copy link

I was thinking gptcommit could hook into the merge commit to summarize a PR. Is that possible? It looks like it's the exact same hook you're already using, but the first argument is merge instead of commit.

I'd take the summary and paste it into my PR before aborting the merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants