Skip to content

Fix config validation message typo#13515

Merged
nicolo-ribaudo merged 2 commits intobabel:mainfrom
jaeseokk:config-validation-message-typo
Jun 29, 2021
Merged

Fix config validation message typo#13515
nicolo-ribaudo merged 2 commits intobabel:mainfrom
jaeseokk:config-validation-message-typo

Conversation

@jaeseokk
Copy link
Copy Markdown
Contributor

@jaeseokk jaeseokk commented Jun 28, 2021

Q                       A
Fixed Issues?
Patch: Bug Fix? Yes
Major: Breaking Change?
Minor: New Feature?
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

Fixed config option names in validation message preset/plugin -> presets/plugins.

@babel-bot
Copy link
Copy Markdown
Collaborator

babel-bot commented Jun 28, 2021

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/47135/

@codesandbox-ci
Copy link
Copy Markdown

codesandbox-ci bot commented Jun 28, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit b8d5987:

Sandbox Source
babel-repl-custom-plugin Configuration
babel-plugin-multi-config Configuration

Copy link
Copy Markdown
Member

@nicolo-ribaudo nicolo-ribaudo left a comment

Choose a reason for hiding this comment

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

Good catch!

}]
]
To be a valid preset, its name and options should be wrapped in a pair of brackets"
To be a valid presets, its name and options should be wrapped in a pair of brackets"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Actually, it's better not to have the s here. Maybe you could edit the checkNoUnwrappedItemOptionPairs adding the s for the "Maybe you meant to use" section, rather than changing it in the parameter?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

good point!
I fixed it as you said.

b8d5987

@nicolo-ribaudo nicolo-ribaudo added area: errors PR: Polish 💅 A type of pull request used for our changelog categories labels Jun 28, 2021
Copy link
Copy Markdown
Member

@nicolo-ribaudo nicolo-ribaudo left a comment

Choose a reason for hiding this comment

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

Thanks!

@nicolo-ribaudo nicolo-ribaudo merged commit 6df0b7c into babel:main Jun 29, 2021
@jaeseokk jaeseokk deleted the config-validation-message-typo branch June 29, 2021 07:10
nicolo-ribaudo pushed a commit to nicolo-ribaudo/babel that referenced this pull request Jul 30, 2021
* Fix config validation message typo

* Fix `checkNoUnwrappedItemOptionPairs` type param
@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Sep 29, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: errors outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Polish 💅 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants