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

feat: mdoc-support #2054

Merged
merged 18 commits into from
Oct 24, 2024

fix: feedback

2738270
Select commit
Loading
Failed to load commit list.
Merged

feat: mdoc-support #2054

fix: feedback
2738270
Select commit
Loading
Failed to load commit list.
DCO-2 / DCO required action Oct 22, 2024 in 0s

Check failed

There are 5 commits incorrectly signed off, the check did not pass.

Summary


Sha Message Pass or fail reason
🟢 8c14678 feat: mdoc alpha Valid sign-off found
🟢 1be0fbf feat: fix: some nits Valid sign-off found
🟢 2fcde60 docs(changeset): feat: mdoc-support Valid sign-off found
🟢 8728d37 Merge branch 'main' of github.com:openwallet-found... Skipped: sign-off not required in merge commit
🟢 95eb0db fix: some nits Valid sign-off found
🟢 6282482 fix(x509): correctly parse uncompressed P256 keys Valid sign-off found
🟢 0dd4621 Merge branch 'main' of github.com:openwallet-found... Skipped: sign-off not required in merge commit
🔴 7ccf8e6 fix: some more nits Sign-off not found
🟢 10b7c2b Merge branch 'mdoc-new' of github.com:auer-martin/... Skipped: sign-off not required in merge commit
🟢 c3e186c chore(x509): remove key prefix for buggy compressi... Valid sign-off found
🟢 fce2553 Merge branch 'mdoc-new' of github.com:auer-martin/... Skipped: sign-off not required in merge commit
🔴 2fffb5f fix: incorporate some feedback Sign-off not found
🔴 911a504 fix: nits Sign-off not found
🔴 4daffbc fix: update lockfile Sign-off not found
🔴 65acce7 fix: remove unused import Sign-off not found
🟢 f08fa37 refactor: update HolderInquirer to use issuerSigne... Valid sign-off found
🟢 f9961e0 fix: tests Valid sign-off found
🟢 2738270 fix: feedback Valid sign-off found

Errors details

Sign-off not found

No sign-off was found in the commit message. This usually means that the author or committer of this commit failed to include a Signed-off-by line in the commit message. In some cases, this error can also be raised if the sign-off is not in the correct format.

To avoid having pull requests blocked in the future, always include a Signed-off-by: User1 <user1@email.test> line in every commit message. You can also do this automatically by using the -s flag (i.e., git commit -s).

How to fix missing or invalid sign-offs

Option 1: add remediation commit

Remediation commits are not allowed for this repository. For more details about how to enable them, please see the documentation.

Option 2: fix commits without sign-off

Rebase the branch

If you have a local git environment and meet the criteria below, one option is to rebase the branch and add your Signed-off-by lines in the new commits. Please note that if others have already begun work based upon the commits in this branch, this solution will rewrite history and may cause serious issues for collaborators (described in the git documentation under "The Perils of Rebasing").

Warning

You should only do this if:

  • You are the only author of the commits in this branch
  • You are absolutely certain nobody else is doing any work based upon this branch
  • There are no empty commits in the branch

To add your Signed-off-by line to every commit in this branch:

  1. Ensure you have a local copy of your branch by checking out the pull request locally via command line.
  2. In your local branch, run: git rebase HEAD~18 --signoff
  3. Force push your changes to overwrite the branch: git push --force-with-lease origin mdoc-new