Skip to content

service:check add null check for validation config#1471

Merged
JakeDawkins merged 3 commits intomasterfrom
evans/add-null-check-for-validationConfig
Aug 27, 2019
Merged

service:check add null check for validation config#1471
JakeDawkins merged 3 commits intomasterfrom
evans/add-null-check-for-validationConfig

Conversation

@evans
Copy link
Copy Markdown
Contributor

@evans evans commented Aug 14, 2019

The validationConfig is set to null when the backend does not perform
any validations. This reformats the markdown output to reflect that

closes #1243

TODO:

  • Update CHANGELOG.md* with your change (include reference to issue & this PR)
  • Make sure all of the significant new logic is covered by tests
  • Rebase your changes on master so that they can be merged easily
  • Make sure all tests and linter rules pass

*Make sure changelog entries note which project(s) has been affected. See older entries for examples on what this looks like.

@JakeDawkins
Copy link
Copy Markdown
Contributor

@evans this looks good to me! Can you update the tests for this?

evans added 3 commits August 21, 2019 11:04
The validationConfig is set to null when the backend does not perform
any validations. This reformats the markdown output to reflect that
@evans evans force-pushed the evans/add-null-check-for-validationConfig branch from 74ab4ce to 3d99cc0 Compare August 21, 2019 19:04
@JakeDawkins JakeDawkins merged commit 30ad29d into master Aug 27, 2019
@JakeDawkins JakeDawkins deleted the evans/add-null-check-for-validationConfig branch August 27, 2019 13:37
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.

apollo service:check --markdown options throw error

2 participants