Skip to content
This repository has been archived by the owner on Jan 24, 2022. It is now read-only.

Add new check command #230

Merged
merged 1 commit into from
Oct 11, 2018
Merged

Add new check command #230

merged 1 commit into from
Oct 11, 2018

Conversation

spalladino
Copy link
Contributor

Adds a new check command that outputs validation errors for all contracts in the project, or allows for a single contract to be specified.

Depends on pull request from branch feature/validate-contract-on-push. Merge after that one.

Copy link
Contributor

@facuspagnuolo facuspagnuolo left a comment

Choose a reason for hiding this comment

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

LGTM, please remove --skip-compile option if #214 gets merged first

@spalladino
Copy link
Contributor Author

spalladino commented Oct 11, 2018

No need to remove, check does not call withPushOptions

@spalladino spalladino added the status:ready-to-merge Order mergify to merge label Oct 11, 2018
Adds a new check command that outputs validation errors for all
contracts in the project, or allows for a single contract to be
specified.
@mergify mergify bot merged commit 4ba5cbf into master Oct 11, 2018
@facuspagnuolo facuspagnuolo deleted the feature/check-command branch October 22, 2018 18:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:ready-to-merge Order mergify to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants