Skip to content

fix(lightning css): align type with lightning documentation#8197

Closed
zackarychapple wants to merge 1 commit intoweb-infra-dev:mainfrom
zackarychapple:main
Closed

fix(lightning css): align type with lightning documentation#8197
zackarychapple wants to merge 1 commit intoweb-infra-dev:mainfrom
zackarychapple:main

Conversation

@zackarychapple
Copy link
Copy Markdown
Contributor

Summary

In the lightning css documentation there is a option to enable drafts. This option was incorrectly named as draft instead of drafts. Updated the typing to the proper name as per the documentation.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Oct 22, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Oct 22, 2024

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 3878720
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/6717b372ac87f900084dc032

@hardfist
Copy link
Copy Markdown
Contributor

even it's a typo, this change is still a breaking change, we can try to be compatible by supporting both draft and drafts and mark draft deprecated and remove it in v2

@chenjiahan
Copy link
Copy Markdown
Member

@GiveMe-A-Name can you take this?

@GiveMe-A-Name
Copy link
Copy Markdown
Member

CI failed, you should run ./x ae update to update publish api.

even it's a typo, this change is still a breaking change, we can try to be compatible by supporting both draft and drafts and mark draft deprecated and remove it in v2

And like hardfist Said, we can support both draft and drafts

@GiveMe-A-Name
Copy link
Copy Markdown
Member

This PR have long time have not progress, I fix it in another PR #8456

@chenjiahan chenjiahan closed this Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants