Skip to content

Document engine requirements in client:check and client:push#1077

Merged
JakeDawkins merged 2 commits intomasterfrom
jake/engine-req
May 13, 2019
Merged

Document engine requirements in client:check and client:push#1077
JakeDawkins merged 2 commits intomasterfrom
jake/engine-req

Conversation

@JakeDawkins
Copy link
Copy Markdown
Contributor

@JakeDawkins JakeDawkins commented Mar 5, 2019

from #1075

Previously, there was no explicit message stating that engine is a requirement for client:check and client:push commands.

It'd be easy to mistake these commands as ones that could be run with a client.service.url or localSchemaFile.

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.

Copy link
Copy Markdown
Contributor

@trevor-scheer trevor-scheer left a comment

Choose a reason for hiding this comment

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

LGTM!

@JakeDawkins JakeDawkins merged commit c13b7d2 into master May 13, 2019
@JakeDawkins JakeDawkins deleted the jake/engine-req branch May 13, 2019 21:39
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.

2 participants