Fix Markdown linter issue in Dot DSL readme

A recent Lint markdown run in the problem-specs repo failed with the following message:

Error: exercises/dot-dsl/description.md:25:88 MD059/descriptive-link-text Link text should be descriptive [Context: "[here]"] https://github.com/DavidAnson/markdownlint/blob/v0.38.0/doc/md059.md
Error: README.md:286:20 MD059/descriptive-link-text Link text should be descriptive [Context: "[here]"] https://github.com/DavidAnson/markdownlint/blob/v0.38.0/doc/md059.md
Error: Failed with exit code: 1

The problem appears to be the Fowler DSL link so I’m proposing a simple change.

Interesting linter. I wonder if the ā€œfound hereā€ should be dropped.

[Learn more about the difference between internal and external DSLs][fowler-dsl].
1 Like

Makes sense to me. The here seems redundant.

Hey @BNAndras,
Did you happen to raise a PR for this yet?
Just asking because this looks like a comparison page rather than an actual pull request.

No, I’ve been waiting to make the PR in case anybody else has input on the matter.

Makes sense! I’m in favour of this change, that persistent linter failure is frustrating to see. :cry:

Alright, I PRed Isaac’s suggestion at Fix linter CI error in `dot-dsl` by BNAndras Ā· Pull Request #2571 Ā· exercism/problem-specifications Ā· GitHub.

Edit - Merged!

2 Likes

Okay, we resolved the linter, but there’s another generic link text warning for problem-specifications/README.md at a5da711b5e2e1ffbbc5ce10ce3471c5570e3e3d7 Ā· exercism/problem-specifications Ā· GitHub.

Perhaps this should be something like this:

[The official Yarn installation directions are available here](https://yarnpkg.com/en/docs/install).

Or something like Follow [the official Yarn installation directions](...)

CI passes now so that PR was merged and then build(deps): bump DavidAnson/markdownlint-cli2-action from 19.1.0 to 20.0.0 by dependabot[bot] Ā· Pull Request #2568 Ā· exercism/problem-specifications Ā· GitHub was rebased and merged.

2 Likes