(fix): Errors related to duplicate use of ansible var 'register' #2675
Check failed
There are 6 commits incorrectly signed off, the check did not pass.
Summary
| Sha | Message | Pass or fail reason | |
|---|---|---|---|
| 🔴 | 3b7431c | Initial commit | Sign-off not found |
| 🔴 | 282ebb9 | Repository Establishment: | Sign-off not found |
| 🟢 | 2bcac98 | Merge upstream Bevel main | Valid sign-off found |
| 🟢 | 0551343 | Remove other platforms and update readme | Valid sign-off found |
| 🟢 | c726a10 | changes for fabric | Valid sign-off found |
| 🟢 | a5d73c4 | fix: remove duplicate register var and fix anchorp... | Valid sign-off found |
| 🟢 | 2540d56 | Merge branch 'develop' into fixes | Skipped: sign-off not required in merge commit |
| 🟢 | 4d845ff | Merge branch 'develop' into fixes | Skipped: sign-off not required in merge commit |
| 🟢 | 5be3390 | Merge branch 'develop' into fixes | Skipped: sign-off not required in merge commit |
| 🟢 | 0a59cac | fixed storageclass | Valid sign-off found |
| 🟢 | cbe04c2 | Fixed add-org | Valid sign-off found |
| 🟢 | 4a4dc0f | updated | Valid sign-off found |
| 🟢 | e6c3d4d | use local user in docker | Valid sign-off found |
| 🟢 | f527cae | updated github location | Valid sign-off found |
| 🟢 | 31b2cc4 | updated chaincode chart | Valid sign-off found |
| 🟢 | 7d9e451 | updated | Valid sign-off found |
| 🟢 | 4fa0beb | updated | Valid sign-off found |
| 🟢 | 56c98c8 | updated | Valid sign-off found |
| 🔴 | 404ca4a | updated | Sign-off not found |
| 🟢 | 4f87c2f | fix external-chaincode | Valid sign-off found |
| 🟢 | 4f7286d | Added external-chaincode lifecycle chart | Valid sign-off found |
| 🟢 | 22c24a0 | Fixed external-chaincode operations automation | Valid sign-off found |
| 🔴 | 0cfcb27 | updated | Sign-off not found |
| 🟢 | cfc167e | tested add-peer | Valid sign-off found |
| 🔴 | cb52b07 | updated | Sign-off not found |
| 🟢 | a5de7ce | Updated | Valid sign-off found |
| 🟢 | 4be854f | fixed add org | Valid sign-off found |
| 🟢 | c6288b7 | Fix fabric multi-org | Skipped: sign-off not required in merge commit |
| 🔴 | ad2bcd4 | merged from deployment environments | Sign-off not found |
| 🟢 | 91daecf | updated samples | Valid sign-off found |
| 🟢 | 14d50cf | Merge remote-tracking branch 'gitlab/development' ... | Skipped: sign-off not required in merge commit |
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:
- Ensure you have a local copy of your branch by checking out the pull request locally via command line.
- In your local branch, run:
git rebase HEAD~31 --signoff - Force push your changes to overwrite the branch:
git push --force-with-lease origin fixes