Skip to content
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

chore(linting): automate tracking of custom Sass functions for stylelint #10313

Conversation

jcfranco
Copy link
Member

@jcfranco jcfranco commented Sep 14, 2024

Related Issue: #10188

Summary

This adds a script that tracks all custom Sass functions and updates the metadata for Stylelint’s function-no-unknown rule, addressing workflow improvements as discussed in #10188 (comment).

Our pre-commit hook will run this script if there is a staged Sass file and will add any changes (if any).

Note: this depends on #10311.

@github-actions github-actions bot added the refactor Issues tied to code that needs to be significantly reworked. label Sep 14, 2024
@jcfranco jcfranco marked this pull request as draft September 14, 2024 02:59
@jcfranco jcfranco changed the title refactor(stylelint): change config to module format to enable more dynamic options chore(linting): automate tracking of custom Sass functions for stylelint Sep 14, 2024
@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label Sep 14, 2024
@jcfranco jcfranco marked this pull request as ready for review September 14, 2024 04:58
Copy link
Member

@benelan benelan left a comment

Choose a reason for hiding this comment

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

A couple notes but after those are resolved it LGTM!

support/updateStylelintCustomSassFunctions.ts Outdated Show resolved Hide resolved
.husky/pre-commit Show resolved Hide resolved
support/updateStylelintCustomSassFunctions.ts Show resolved Hide resolved
support/updateStylelintCustomSassFunctions.ts Outdated Show resolved Hide resolved
@jcfranco jcfranco added the skip visual snapshots Pull requests that do not need visual regression testing. label Sep 16, 2024
@jcfranco jcfranco merged commit 62259d1 into dev Sep 16, 2024
10 of 11 checks passed
@jcfranco jcfranco deleted the jcfranco/10188-dynamically-track-custom-sass-functions-for-linting branch September 16, 2024 21:20
benelan added a commit that referenced this pull request Sep 17, 2024
…estone-estimates

* origin/dev: (997 commits)
  fix: correct non-standard filled icon names (#10309)
  fix(panel): initially closed panel should be hidden (#10308)
  chore(linting): automate tracking of custom Sass functions for stylelint (#10313)
  chore: tidy up demo pages (#10314)
  build(deps): update dependency dayjs to v1.11.13 (#10283)
  build(deps): update dependency jsdom to v24.1.3 (#10298)
  build(deps): update dependency husky to v9.1.6 (#10318)
  build(deps): update angular-cli monorepo to v18.2.4 (#10317)
  docs: update component READMEs (#10316)
  refactor(stylelint): change config to module format to enable more dynamic options (#10311)
  refactor: fixup typescript errors (#10295)
  build(deps): update dependency lint-staged to v15.2.10 (#10302)
  build(deps): update dependency focus-trap to v7.6.0 (#10281)
  build(deps): update dependency husky to v9.1.5 (#10297)
  chore: release next
  feat(checkbox): add component tokens (#10221)
  revert: "chore: set default page size for E2E tests (#10219)" (#10299)
  chore(icons): ensure UI icons follow naming convention (#10292)
  chore: release next
  feat: add model-history, raster-function-history, raster function-template-history, raster-tool-history, tool-history (#10305)
  ...
benelan added a commit that referenced this pull request Sep 19, 2024
* origin/dev:
  fix(shell): fix z-index of slotted popover elements (#10325)
  fix(dialog): only prevent default on escape key when escapeDisabled is true (#10336)
  chore: release next
  feat(carousel): Improve support for high item counts (#10315)
  feat: add content inline right (#10338)
  fix: correct non-standard filled icon names (#10309)
  fix(panel): initially closed panel should be hidden (#10308)
  chore(linting): automate tracking of custom Sass functions for stylelint (#10313)
  chore: tidy up demo pages (#10314)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files. refactor Issues tied to code that needs to be significantly reworked. skip visual snapshots Pull requests that do not need visual regression testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants