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.

Sign up for GitHub

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

Jump to bottom

Check shouldn't be called from Apply #240

Closed
BrianHicks opened this issue Sep 13, 2016 · 1 comment
Closed

Check shouldn't be called from Apply #240

BrianHicks opened this issue Sep 13, 2016 · 1 comment
Assignees
@BrianHicks
Labels
bug
Milestone
0.2.0

Comments

@BrianHicks
Copy link
Contributor

The execution pipeline does this, they should be separate things that return their own reasonably-set fields. This is a precedent we should probably not set.
@Dacode45
Copy link
Contributor

Good to know.

@BrianHicks BrianHicks assigned BrianHicks and unassigned BrianHicks Sep 19, 2016
@BrianHicks BrianHicks added this to the 0.2.0-beta2 milestone Sep 19, 2016
@BrianHicks BrianHicks self-assigned this Sep 20, 2016
@BrianHicks BrianHicks mentioned this issue Sep 20, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@BrianHicks BrianHicks

Labels
bug
Projects
None yet
Milestone
0.2.0
Development

No branches or pull requests
2 participants
@BrianHicks @Dacode45