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

Signed Commits #217

Closed
C0D3-M4513R opened this issue Jan 7, 2022 · 5 comments · Fixed by #221
Closed

Signed Commits #217

C0D3-M4513R opened this issue Jan 7, 2022 · 5 comments · Fixed by #221
Assignees
Labels
enhancement New feature or request

Comments

@C0D3-M4513R
Copy link

C0D3-M4513R commented Jan 7, 2022

Is your feature request related to a problem? Please describe.
I recently got a pull request in a repo of mine, to update the Year in the license as expected.
My issue with that PR was with two things:

  1. My Linter Action for PRs just never completed?
  2. The Commit to the PR was unsigned. I require signed commits though.

Describe the solution you'd like
Add a way, for this github-action to make signed commits.

Describe alternatives you've considered
Not requiring commit signing? idk?

#119 (comment) :

because it would most probably require you to save your private key in GitHub

As a file, or in a config publicly (since workflows cannot use secrets)? - If yes, than this should NEVER be realeased as a feature. Public repos should NEVER have private gpg keys publicly accessible and private repos shouldn't have a need for Licences and signed commits.

@C0D3-M4513R C0D3-M4513R added the enhancement New feature or request label Jan 7, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Jan 7, 2022

Hi there and welcome to this repository!

A maintainer will be with you shortly, but first and foremost I would like to thank you for taking the time to report this issue. Quality is of the highest priority for us, and we would never release anything with known defects. We aim to do our best but unfortunately you are here because you encountered something we didn't expect. Lets see if we can figure out what went wrong and provide a remedy for it.

@FantasticFiasco
Copy link
Owner

Hi @C0D3-M4513R and welcome to the repo!

Regarding the linter, I looked at your repo and especially the linter workflow, and found that this action isn't used in that workflow, thus I don't think they have anything in common. The problem with the linter is probably unrelated with this action.

Regarding signed commits, I will start to investigate it. To ease your mind, it would of course not involve publicly accessible secrets, but instead rely on GitHub Secrets, configured on the repo or organization. These secrets are available on branches and pull requests opened from the same repo, but will not be available on pull requests opened from forks.

@FantasticFiasco
Copy link
Owner

A heads up regarding the progress. The implementation is done, tests and documentation remains.

@FantasticFiasco
Copy link
Owner

The feature has now been released and you've been added to the list of contributors in README.md.

Thanks for the feature request!

@C0D3-M4513R
Copy link
Author

I saw, that I've been added. Thank you, for making this action.

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

Successfully merging a pull request may close this issue.

2 participants