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

docs: fixed link readme #9625

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

stopwar2025
Copy link

No description provided.

Copy link

netlify bot commented Mar 17, 2025

👷 Deploy Preview for chef-habitat processing.

Name Link
🔨 Latest commit 0d5ecbb
🔍 Latest deploy log https://app.netlify.com/sites/chef-habitat/deploys/67d8269c1715dd00086a8b68

Copy link
Contributor

Hello stopwar2025! Thanks for the pull request!

Here is what will happen next:

  1. Your PR will be reviewed by the maintainers.
  2. If everything looks good, one of them will approve it, and your PR will be merged.

Thank you for contributing!

@mwrock
Copy link
Contributor

mwrock commented Mar 18, 2025

@stopwar2025 thanks for this. Could you please sign your commit (see https://github.com/chef/chef/blob/main/CONTRIBUTING.md#developer-certification-of-origin-dco). Basically you need to add an -s to the end of your git commit.

@agadgil-progress agadgil-progress dismissed their stale review March 24, 2025 03:49

Require signing commits

@stopwar2025
Copy link
Author

stopwar2025 commented Mar 24, 2025

I don't understand how to do this, since I make corrections through the documentation editing function on GitHub without a terminal.

@mwrock
Copy link
Contributor

mwrock commented Mar 24, 2025

@stopwar2025 yeah this has to be done on the command line. Unfortunately we cannot accept PRs from commits submitted through the github ui unless there is now a way to sign those commits that I am not aware of.

@stopwar2025
Copy link
Author

I'm sorry, I understand you.

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.

3 participants