Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

feat: add signature option #120

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

Merged
merged 1 commit into from
May 16, 2023
Merged

Conversation

stefanbuck
Copy link
Contributor

@stefanbuck stefanbuck commented Feb 22, 2023

Fixes #118 by introducing a new property signature.

I didn't want to introduce another dependency to octokit-plugin-create-pull-request. Therefore signature is a function which allows the caller to implement the actual commit signing however it suits best to them.

@stefanbuck stefanbuck marked this pull request as ready for review February 22, 2023 20:21
@stefanbuck stefanbuck force-pushed the feat-accept-signature branch from 64f7c90 to 00f31b4 Compare February 22, 2023 20:55
@stefanbuck
Copy link
Contributor Author

Not urgent at all, but a friendly reminder to take a look at this whenever you have time @gr2m

Copy link
Owner

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

Very sorry for the delay 😞 I tested it locally and it CI tests pass

@gr2m gr2m merged commit 3fe11f6 into gr2m:main May 16, 2023
@github-actions
Copy link

🎉 This PR is included in version 4.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

Feature request: Allow commits to be signed
2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.