-
Notifications
You must be signed in to change notification settings - Fork 40
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
feat(cli): support json configuration for tokens create
#2847
Conversation
🦋 Changeset detectedLatest commit: 9a91153 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Preview deployments for this pull request: Storybook - |
807abb5
to
7561391
Compare
Coverage Report
File CoverageNo changed files found. |
0489d12
to
e745bf4
Compare
a776f8c
to
1fd3e22
Compare
1fd3e22
to
afc6dc1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added missing changeset.
I think this is a good start. We don't need to promote this feature yet, but test it internally for now and fine tune it.
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to next, this PR will be updated.⚠️ ⚠️ ⚠️ ⚠️ ⚠️ ⚠️ `next` is currently in **pre mode** so this branch has prereleases rather than normal releases. If you want to exit prereleases, run `changeset pre exit` on `next`.⚠️ ⚠️ ⚠️ ⚠️ ⚠️ ⚠️ # Releases ## @digdir/[email protected] ### Minor Changes - Add JSON schema for CLI config file, which enables editor hints. To use it, do something like this: ([#2972](#2972)) ```jsonc { "$schema": "node_modules/@digdir/designsystemet/dist/config.schema.json", // ...config options here... } ``` - cli: better error messages for wrong or missing options ([#2970](#2970)) ### Patch Changes - tokens: fix wrong reference for focus color. ([#2989](#2989)) - refactor: Changed border-radius token to dimension type ([#2990](#2990)) - Add json config file support for `tokens create` ([#2847](#2847)) - Changed focus color to use neutral instead of accent color ([#2989](#2989)) - tokens build: fix invalid values in --ds-size calculations ([#2975](#2975)) - outDir option was required, but it should have a default value ([#2969](#2969)) ## @digdir/[email protected] ### Patch Changes - Helptext: Remove component ([#2956](#2956)) - Field: Don't show as disabled when option is disabled ([#2980](#2980)) - Changed focus color to use neutral instead of accent color ([#2989](#2989)) - Tabs: Rework component CSS ([#2991](#2991)) ## @digdir/[email protected] ### Patch Changes - Input: Add whitelist for `type` ([#2974](#2974)) - Helptext: Remove component ([#2956](#2956)) - Textfield: Add whitelist for `type` ([#2974](#2974)) - Tabs: Content will get focus when it has no focusable elements ([#2991](#2991)) ## @digdir/[email protected] ### Patch Changes - Changed focus color to use neutral instead of accent color ([#2989](#2989)) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Closes #2846
Some things to note:
--json
was not used or default config was not found, but this doesn't seem to be possiblegetExplicitOptionOnly()
has been added for this purpose.