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(gatsby-plugin-flow): Add pluginOptionsSchema validation #27599

Conversation

MichaelDeBoey
Copy link
Contributor

Description

Just like @mxstbr did in #27356

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Oct 22, 2020
@wardpeet
Copy link
Contributor

wardpeet commented Oct 22, 2020

Thanks! Can you add tests?

@MichaelDeBoey
Copy link
Contributor Author

@wardpeet I figured since there's not really anything to test and #27356 also doesn't have tests, I didn't need to add them.
But if you want, I can add them for you

@wardpeet
Copy link
Contributor

please add them cause when we add options we should update the tests, even if it's an empty object.

@ascorbic ascorbic added topic: plugins Related to plugin system, themes & catch-all for plugins that don't have a label and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Oct 22, 2020
@ascorbic ascorbic requested a review from mxstbr October 22, 2020 15:46
@MichaelDeBoey MichaelDeBoey force-pushed the plugin-flow/add-pluginOptionsSchema-validation branch 2 times, most recently from 9899009 to 7936fe0 Compare October 22, 2020 23:44
@mxstbr
Copy link
Contributor

mxstbr commented Oct 27, 2020

👋 @MichaelDeBoey thank you so much for tackling this, love it! 💯

I'm working on the last bits and pieces for plugin option validation to be stable right now, after which I'm going to take a look at shipping these PRs 👍 #27600 #27601 #27602

@MichaelDeBoey
Copy link
Contributor Author

@mxstbr Take your time, there's no rush 😉

@MichaelDeBoey MichaelDeBoey force-pushed the plugin-flow/add-pluginOptionsSchema-validation branch from 7936fe0 to 4adddc8 Compare October 28, 2020 12:56
@MichaelDeBoey MichaelDeBoey requested a review from a team as a code owner October 28, 2020 12:56
mxstbr
mxstbr previously approved these changes Nov 4, 2020
Copy link
Contributor

@mxstbr mxstbr left a comment

Choose a reason for hiding this comment

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

Thank you!!

mxstbr
mxstbr previously approved these changes Nov 4, 2020
@mxstbr mxstbr added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label Nov 4, 2020
@MichaelDeBoey MichaelDeBoey force-pushed the plugin-flow/add-pluginOptionsSchema-validation branch from 680d743 to 1531e15 Compare November 9, 2020 01:27
mxstbr
mxstbr previously approved these changes Nov 13, 2020
Copy link
Contributor

@mxstbr mxstbr left a comment

Choose a reason for hiding this comment

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

Same flakey tests as always that are failing, unrelated to this PR: #27781. Shipping!

@MichaelDeBoey
Copy link
Contributor Author

@wardpeet @mxstbr I think this one can be merged, no? 🤔

Copy link
Contributor

@LekoArts LekoArts left a comment

Choose a reason for hiding this comment

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

Thanks!

@gatsbybot gatsbybot merged commit 0668750 into gatsbyjs:master Apr 21, 2021
@MichaelDeBoey MichaelDeBoey deleted the plugin-flow/add-pluginOptionsSchema-validation branch April 21, 2021 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: merge on green Gatsbot will merge these PRs automatically when all tests passes topic: plugins Related to plugin system, themes & catch-all for plugins that don't have a label
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants