Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

Adding a linter with a basic configuration #17

Merged
merged 5 commits into from
May 22, 2024

Conversation

gkunz
Copy link
Contributor

@gkunz gkunz commented May 9, 2024

This adds a markdown linter as a GitHub action to ensure that the texts which are being added use consistent and correct markdown syntax.

@myteron
Copy link
Contributor

myteron commented May 10, 2024

There are two formatting issues on the main /readme.md that would block deliveries.

  1. forced width of tables, looks horrible without
    would need a solution or exclude MD033 inline haml and MD045 missing alt string on image
    Note that in 'theory' MD supports dots for spaces but is practically not working. Neither did I find anything else working.

  2. Forced newlines in cells.
    would need a solution or exclude MD033 inline html
    In theory MD supports .. or \ for forced nl but in practice table rows required a continues line, so can't newline. Besides \ +nl or .. + nl does not work anyways as it seems.

@myteron myteron self-assigned this May 10, 2024
@BartyBoi1128 BartyBoi1128 merged commit 3e85274 into Ericsson:main May 22, 2024
1 check failed
@gkunz gkunz deleted the adding-linter branch May 23, 2024 10:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants