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

feat(scully): add exit by default when have a plugin error and cli option #496

Merged
merged 3 commits into from
Apr 23, 2020

Conversation

jorgeucano
Copy link
Contributor

…he flag for dont use this funct

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Other... Please describe:

What is the current behavior?

Issue Number: #492 #493

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@jorgeucano
Copy link
Contributor Author

@spencerb02 this is the other PR for exit when have the error in the plugin

Copy link

@spencerb02 spencerb02 left a comment

Choose a reason for hiding this comment

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

Looks good. I like that exiting the process is the default behavior. Well done!

@SanderElias
Copy link
Contributor

@jorgeucano The code was really good. However, I moved it a bit around. Now it works on all plugins and not only on render ones.

Copy link
Contributor

@SanderElias SanderElias left a comment

Choose a reason for hiding this comment

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

LGTM

@jorgeucano jorgeucano merged commit 2ecc216 into master Apr 23, 2020
@jorgeucano jorgeucano deleted the fix/492-493 branch April 23, 2020 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants