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

[feature] #306: Validate links post-build #405

Merged
merged 11 commits into from
Sep 14, 2023

Conversation

0x009922
Copy link
Contributor

Closes #306

@0x009922 0x009922 self-assigned this Sep 13, 2023
@vercel
Copy link

vercel bot commented Sep 13, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
iroha-2-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 14, 2023 2:29am

Signed-off-by: Dmitry Balashov <[email protected]>
Signed-off-by: Dmitry Balashov <[email protected]>
Copy link
Contributor

@6r1d 6r1d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

src/guide/advanced/running-iroha-on-bare-metal.md Outdated Show resolved Hide resolved
src/guide/blockchain/data-model.md Outdated Show resolved Hide resolved
src/guide/build.md Outdated Show resolved Hide resolved
src/guide/configure/peer-configuration.md Outdated Show resolved Hide resolved
src/guide/glossary.md Outdated Show resolved Hide resolved
0x009922 and others added 2 commits September 14, 2023 09:27
Co-authored-by: William Richter <[email protected]>
Signed-off-by: 0x009922 <[email protected]>
@0x009922
Copy link
Contributor Author

@WRRicht3r, this PR is focused on functional changes and related documentation fixes. While I appreciate your suggestions on doc strings, they are technically outside the scope of this PR. I have included them this time, but in the future, I would prefer to keep PRs more granular.

@0x009922 0x009922 merged commit 9bc0f28 into hyperledger-iroha:main Sep 14, 2023
4 checks passed
@0x009922 0x009922 deleted the 306-validate-links branch September 14, 2023 08:02
6r1d pushed a commit to 6r1d/iroha-2-docs that referenced this pull request Jan 8, 2024
…er-iroha#405)

* [feature]: implement links validation, without pub path

Signed-off-by: Dmitry Balashov <[email protected]>

* [feature]: resolve links with pub path

Signed-off-by: Dmitry Balashov <[email protected]>

* [feature]: refine reporting

Signed-off-by: Dmitry Balashov <[email protected]>

* [ci]: validate links

Signed-off-by: Dmitry Balashov <[email protected]>

* [refactor]: chores

Signed-off-by: Dmitry Balashov <[email protected]>

* [chore]: fix format

Signed-off-by: Dmitry Balashov <[email protected]>

* [ci]: workaround with FORCE_COLOR

Signed-off-by: Dmitry Balashov <[email protected]>

* [ci]: use FORCE_COLOR in gh-pages-deploy too

Signed-off-by: Dmitry Balashov <[email protected]>

* [docs]: fix broken links

Signed-off-by: Dmitry Balashov <[email protected]>

* Apply suggestions from code review

Co-authored-by: William Richter <[email protected]>
Signed-off-by: 0x009922 <[email protected]>

* Update src/guide/advanced/running-iroha-on-bare-metal.md

Co-authored-by: William Richter <[email protected]>
Signed-off-by: 0x009922 <[email protected]>

---------

Signed-off-by: Dmitry Balashov <[email protected]>
Signed-off-by: 0x009922 <[email protected]>
Co-authored-by: William Richter <[email protected]>
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.

Validate anchor links
3 participants