Skip to content

Conversation

@martinyde
Copy link

@martinyde martinyde commented Jun 30, 2025

Link to ticket

https://leantime.itkdev.dk/404/#/tickets/showTicket/4829

Description

  • Apply itkdev template
  • Apply coding standards
  • Removed composer scripts
  • Updated Taskfiles

Additional comments or questions

The old site apparently couldn't be installed due to config setup. Removing ref: ${{ github.base_ref }} would allow the site action to pass, which suggests that after the update the site would install since config is changed.

@martinyde martinyde mentioned this pull request Jul 2, 2025
@martinyde martinyde requested a review from rimi-itk July 2, 2025 12:13
Copy link

@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.

A few questions asked.

docker network create frontend
- run: |
docker compose run --rm prettier 'web/profiles/custom/os2loop/**/*.{css,scss}' --check
Copy link

Choose a reason for hiding this comment

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

Would

Suggested change
docker compose run --rm prettier 'web/profiles/custom/os2loop/**/*.{css,scss}' --check
docker compose run --rm prettier 'web/*/custom/**/css/**/*.{css,scss}' --check

work here? If so, we could make a small change in https://github.com/itk-dev/devops_itkdev-docker/blob/develop/github/workflows/drupal/styles.yaml#L36 for (more) a general solution.

Copy link
Author

Choose a reason for hiding this comment

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

Afraid not. The first * could work but the styling assets are not contained in a specific css folder

@martinyde martinyde requested a review from rimi-itk July 2, 2025 13:55
@martinyde martinyde merged commit ca6a55e into feature/d-11-prep Jul 7, 2025
11 of 12 checks passed
@martinyde martinyde deleted the feature/apply-itkdev-template branch July 7, 2025 07:41
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.

3 participants