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

fix: dci lint errors #514 #516

Merged
merged 2 commits into from
Feb 1, 2021

Conversation

petermetz
Copy link
Contributor

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]

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]>
@petermetz petermetz added documentation Improvements or additions to documentation good-first-issue Good for newcomers Developer_Experience Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. labels Jan 26, 2021
@petermetz petermetz added this to the v0.4.0 milestone Jan 26, 2021
@petermetz petermetz enabled auto-merge (rebase) January 26, 2021 21:42
Copy link
Contributor

@jonathan-m-hamilton jonathan-m-hamilton left a 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
Copy link
Contributor

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

Copy link
Contributor Author

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

@takeutak Sure thing! Here's the commit: 2e13877

petermetz added a commit to petermetz/cacti that referenced this pull request Jan 29, 2021
@petermetz petermetz force-pushed the fix/dci-lint-errors-514 branch from 47b08d3 to 2e13877 Compare January 29, 2021 02:47
@takeutak takeutak self-requested a review February 1, 2021 22:53
@petermetz petermetz disabled auto-merge February 1, 2021 23:41
@petermetz petermetz merged commit 9dbb10d into hyperledger-cacti:main Feb 1, 2021
@petermetz petermetz deleted the fix/dci-lint-errors-514 branch February 1, 2021 23:42
ghost pushed a commit to kikoncuo/cactus that referenced this pull request Feb 4, 2021
ghost pushed a commit to kikoncuo/cactus that referenced this pull request Feb 4, 2021
Fixes hyperledger-cacti#516 (comment)

Signed-off-by: Peter Somogyvari <[email protected]>
Signed-off-by: Jordi Giron <[email protected]>
ghost pushed a commit to kikoncuo/cactus that referenced this pull request Feb 4, 2021
Fixes hyperledger-cacti#516 (comment)

Signed-off-by: Peter Somogyvari <[email protected]>
Signed-off-by: Jordi Giron <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Developer_Experience documentation Improvements or additions to documentation good-first-issue Good for newcomers Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: dci lint errors
3 participants