Skip to content

Conversation

@martinyde
Copy link
Contributor

@martinyde martinyde commented Jun 25, 2025

Add markdown lint and prettier services to all templates

@martinyde martinyde requested review from rimi-itk and turegjorup June 25, 2025 11:30
Comment on lines 82 to 99
markdownlint:
image: itkdev/markdownlint
profiles:
- dev
volumes:
- ./:/md

prettier:
# Prettier does not (yet, fcf.
# https://github.com/prettier/prettier/issues/15206) have an official
# docker image.
# https://hub.docker.com/r/jauderho/prettier is good candidate (cf. https://hub.docker.com/search?q=prettier&sort=updated_at&order=desc)
image: jauderho/prettier
profiles:
- dev
volumes:
- ./:/work

Copy link
Contributor

Choose a reason for hiding this comment

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

It may be a good idea to move this to the end of the file, i.e. after “the real” project services, to make it clear that these services are “only” used for code checks, e.g.

  

  memcached:
    

  # Code checks tools
  markdownlint:
    

  

Copy link
Contributor

@rimi-itk rimi-itk left a comment

Choose a reason for hiding this comment

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

@martinyde martinyde merged commit 0f77b2f into develop Jun 27, 2025
18 checks passed
@martinyde martinyde deleted the hotfix/add-markdownlint-prettier branch June 27, 2025 08:55
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.

4 participants