-
Notifications
You must be signed in to change notification settings - Fork 295
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
fix: dci lint errors #514 #516
fix: dci lint errors #514 #516
Conversation
Also adds a .dcilintignore file to specify which files are not possible to change without breaking third party dependencies. TODO: If and when these dependencies get swapped out or they release new versions with inclusive language, we'll need to update them accordingly and the ignore file as well. Fixes hyperledger-cacti#514 Signed-off-by: Peter Somogyvari <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
||
- c.f.: https://hyperledger-fabric.readthedocs.io/en/master/install.html#install-samples-binaries-and-docker-images | ||
- c.f.: https://hyperledger-fabric.readthedocs.io/en/release-2.2/install.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you change the URL to the following URL?
https://hyperledger-fabric.readthedocs.io/en/release-1.4/install.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you change the URL to the following URL?
https://hyperledger-fabric.readthedocs.io/en/release-1.4/install.html
Fixes hyperledger-cacti#516 (comment) Signed-off-by: Peter Somogyvari <[email protected]>
47b08d3
to
2e13877
Compare
Fixes hyperledger-cacti#516 (comment) Signed-off-by: Peter Somogyvari <[email protected]>
Fixes hyperledger-cacti#516 (comment) Signed-off-by: Peter Somogyvari <[email protected]> Signed-off-by: Jordi Giron <[email protected]>
Fixes hyperledger-cacti#516 (comment) Signed-off-by: Peter Somogyvari <[email protected]> Signed-off-by: Jordi Giron <[email protected]>
Also adds a .dcilintignore file to specify which files are
not possible to change without breaking third party dependencies.
TODO: If and when these dependencies get swapped out or they
release new versions with inclusive language, we'll need to
update them accordingly and the ignore file as well.
Fixes #514
Signed-off-by: Peter Somogyvari [email protected]