@sketch-hq/sketch-assistant-core-rules

sketch-assistant-core-rules

💁‍♀️ This is a "rules only" Assistant, meaning it contains only rule implementations and no configuration. Adding this Assistant to Sketch won't do anything because rules need to be activated in configuration before they will report anything.

Use this Assistant by extending from it in your own Assistant, and adding configuration and/or additional rule implementations. To find out how to do this and more head to the Assistant Developer Documentation.

⚠️ This package is in pre-release mode using the next tag.

Available rules

This Assistant exports the following rules. Click the links to view their documentation, configuration options and implementation.

• artboards-grid
• artboards-layout
• artboards-max-ungrouped-layers
• borders-no-disabled
• debug-all-options
• debug-i18n
• debug-throws-error
• exported-layers-normal-blend-mode
• fills-no-disabled
• groups-max-layers
• groups-no-empty
• groups-no-redundant
• groups-no-similar
• images-no-outsized
• images-no-undersized
• inner-shadows-no-disabled
• layer-styles-no-dirty
• layer-styles-prefer-library
• layer-styles-prefer-shared
• layers-no-hidden
• layers-no-loose
• layers-subpixel-positioning
• name-pattern-artboards
• name-pattern-groups
• name-pattern-images
• name-pattern-pages
• name-pattern-shapes
• name-pattern-symbols
• name-pattern-text
• result-messages-include
• shadows-no-disabled
• shared-styles-no-unused
• symbols-no-unused
• symbols-prefer-library
• text-styles-no-dirty
• text-styles-prefer-library
• text-styles-prefer-shared

Development

The following section of the readme only relates to developing the Assistant, not using it in your own projects.

Requirements

• Node
• Yarn

Scripts

Interact with the tooling in this repository via the following scripts.

Script	Description
yarn build	Builds the Assistant to the dist folder
yarn format-check	Checks the formatting with prettier
yarn i18n:extract	Extracts strings from the JS source to PO files
yarn test	Runs the Jest tests
yarn type-check	Typecheck the TypeScript

Workflows

Conventional commits

Try and use the conventional commits convention when writing commit messages.

Working on rules

Use a TDD approach, and the existing rules as a guide.

Rule functions can run in Node against plain Sketch file JSON, so the Mac Sketch app isn't required while actively developing rule logic.

When creating a new rule in this repository:

1. Copy and paste one of the existing rule folders in src/ as a starting point.
2. Add the new rule to the Assistant definition in src/index.ts.
3. With Sketch, create some example .sketch files to use as test fixtures. At a minimum these should demonstrate two scenarios - one where you'd expect your rule to report violations, and one where it shouldn't report any
4. While developing your rule and its tests use Jest in watch mode: yarn test --watch.
5. If you've added new strings then follow the internationalization workflow too.
6. Add changeset for your rule, open a Pull Request and once merged release it

Releases

This repository uses Atlassian Changesets to automate the npm release process. Read the docs for more information, but the top-level summary is:

• A GitHub Action maintains a permanently open PR that when merged will publish the package to npm with the latest changes and an automatically determined semver.
• If the work you do in a PR should affect the next release, then you need to commit a "changeset" to the repository together with the rest of your code changes - do this by running yarn changeset. You'll be asked to provide a change type (major, minor or patch) and a message.

Internationalization

This Assistant requires internationalization since it's an official Sketch Assistant.

Translating your own Assistants is completely optional however.

Internationalization is handled by LinguiJS, and follows their guides for handling plain JavaScript projects.

The following locales are currently supported in the Sketch Mac app and this project:

• en (American English)
• zh-Hans (Chinese Simplified)

When ready to perform a round of translation, for example when there are new untranslated strings in the repository, perform the following workflow:

1. Run yarn i18n:extract which will update the .po files in src/locale/ with the new strings used throughout the source code.
2. PR and merge these changes to master.
3. Translation thereafter happens via CrowdIn automation.

Changelog

3.1.0-next.3

Patch Changes

• 461d018: Add copy changes from review.
• 6e8ba28: Resolves the bug with quarter subpixel positions