Skip to content

Fix component template#345

Merged
simu merged 3 commits intomasterfrom
fix/component-template
Jul 6, 2021
Merged

Fix component template#345
simu merged 3 commits intomasterfrom
fix/component-template

Conversation

@simu
Copy link
Member

@simu simu commented Jul 6, 2021

  • Use correct make target for linting documentation
  • Don't emit trailing empty line in component/<component>.yml when creating a component without post-processing filters
  • Adds tests which run make lint on all variants of the rendered component template

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Update tests.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog

@simu simu added the bug Something isn't working label Jul 6, 2021
@simu

This comment has been minimized.

@simu simu force-pushed the fix/component-template branch from 0328c8b to 0506371 Compare July 6, 2021 11:13
simu added 3 commits July 6, 2021 14:21
Use correct make target `lint_adoc` for linting documentation with
Vale.
Make sure the files resulting from the component template don't produce
lint errors without any modifications.
This ensures that all variants of the rendered component-template
actually pass the lints which are provided out of the box.
@simu simu force-pushed the fix/component-template branch from 0506371 to db751c5 Compare July 6, 2021 12:21
@simu simu marked this pull request as ready for review July 6, 2021 12:22
@simu simu requested review from ccremer and corvus-ch July 6, 2021 12:41
@simu simu merged commit 1788a4a into master Jul 6, 2021
@simu simu deleted the fix/component-template branch July 6, 2021 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants