Skip to content
This repository was archived by the owner on Apr 4, 2024. It is now read-only.

add an info pass for md linting#64

Merged
NicolasMassart merged 2 commits into
Consensys:masterfrom
NicolasMassart:add-an-info-pass-for-md-linting
Feb 25, 2020
Merged

add an info pass for md linting#64
NicolasMassart merged 2 commits into
Consensys:masterfrom
NicolasMassart:add-an-info-pass-for-md-linting

Conversation

@NicolasMassart
Copy link
Copy Markdown
Member

Pull Request Description

Adding an info pass that's not breaking the build

Line length check is now only in the info pass as we want them not
to break the build.

The breaking checks are on another run in the same job

Two artefacts are now produced by the CI job: markdownlint.out
and markdownlint_info.out

NOTE: this is the exact same as besu-eth/besu-docs#205 except that it uses the common submodule

Adding an info pass that's not breaking the build

Line length check is now only in the info pass as we want them not
to break the build.

The breaking checks are on another run in the same job

Two artefacts are now produced by the CI job: markdownlint.out
and markdownlint_info.out
@NicolasMassart NicolasMassart requested a review from a team February 25, 2020 15:20
@NicolasMassart NicolasMassart self-assigned this Feb 25, 2020
@NicolasMassart NicolasMassart added the DocOps Related to documentation system, tools and CI label Feb 25, 2020
Copy link
Copy Markdown
Contributor

@bgravenorst bgravenorst left a comment

Choose a reason for hiding this comment

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

Looks good

@NicolasMassart NicolasMassart merged commit e1969f4 into Consensys:master Feb 25, 2020
@NicolasMassart NicolasMassart deleted the add-an-info-pass-for-md-linting branch August 11, 2020 14:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

DocOps Related to documentation system, tools and CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants