Skip to content

Adopt JS tools stylelint, depcruise, styleguidist to include new packages directory #7808

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Feb 13, 2025

Conversation

alexander-schranz
Copy link
Member

@alexander-schranz alexander-schranz commented Feb 13, 2025

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets fixes #
Related issues/PRs #7672
License MIT
Documentation PR sulu/sulu-docs#

What's in this PR?

Adopt stylelint, depcruise and styleguidist to include new packages directory.

Why?

The Snippet package has JS and we require to add validation for that.

Todo

  • Tests (work out of box)
  • Flow (work out of box)
  • Depcruise: updated
  • dependency-cruiser.json
  • Stylelint: updated
  • Styleguidist

@alexander-schranz alexander-schranz changed the title Adopt stylelint and depcruise to include new packages directory Adopt JS tools stylelint, depcruise, styleguidist to include new packages directory Feb 13, 2025
@alexander-schranz alexander-schranz marked this pull request as ready for review February 13, 2025 10:19
@alexander-schranz alexander-schranz added this to the Release 3.0 milestone Feb 13, 2025
@Prokyonn Prokyonn merged commit b29ae09 into sulu:3.0 Feb 13, 2025
8 of 9 checks passed
@alexander-schranz alexander-schranz deleted the enhancement/packages-js-include branch February 13, 2025 12:51
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.

2 participants