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

Conditions #46

Merged
merged 17 commits into from
Oct 29, 2016
Merged

Conditions #46

merged 17 commits into from
Oct 29, 2016

Conversation

bkeepers
Copy link
Contributor

@bkeepers bkeepers commented Oct 29, 2016

on pull_request.opened
if @issue.title matches "^WIP"     # Title starts with "WIP"
or @issue.body matches "- \[ ]\"   # Body has unchecked checkboxes
then label(wip);

View docs

This pull request is the beginning of support for "conditions". At the moment, conditions support built in functions (like labeled(foo)) and logical expressions (like a and b). Attributes of the event payload can be accessed with the @ syntax (like @sender.login matches "bot$")

  • Document all the conditions
  • Figure out "not" syntax punting for now
  • Fix order of precedence for expressions

@bkeepers bkeepers changed the title WIP: Conditions Conditions Oct 29, 2016
@bkeepers
Copy link
Contributor Author

💥 Just tested that this works!

wip__testing_conditions_ _issue__36_ _bkeepers-inc_test

@bkeepers bkeepers merged commit 32ecd73 into master Oct 29, 2016
@bkeepers bkeepers deleted the conditions branch October 29, 2016 17:47
@bkeepers
Copy link
Contributor Author

And here's a public demo of it: https://github.com/bkeepers/test/pull/34

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.

None yet

1 participant