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

Parser #44

Merged
merged 17 commits into from Oct 20, 2016
Merged

Parser #44

merged 17 commits into from Oct 20, 2016

Conversation

bkeepers
Copy link
Contributor

@bkeepers bkeepers commented Oct 17, 2016

This uses peg.js to implement a parser for a new configuration grammar. This new grammar currently supports all of the options that are currently available. Here are some examples:

on issues.opened then close and lock;
on pull_requests.opened then assign(bkeepers) and label(triage);
on issues.opened and pull_request.opened then close;
on issues.opened then comment("Hello World!");
  • Implement parser for basic on…then syntax
  • Use parser in place of yaml-based config
  • Update config docs

Fixes #35

* origin/master:
  Document react action
  Remove 'why' section
  No longer aspirational
  Remove unimplemented examples from README
  Fix then syntax
  Update not at top of readme
  Move aspirational examples to separate file
  Remove unimplemented merge action
  Remove unimplemented from_file option
  Remove unimplemented payload conditions
  Remove interit_from syntax
...because @arfon said so
@bkeepers bkeepers changed the title WIP: Parser Parser Oct 20, 2016
@bkeepers bkeepers merged commit 24fa404 into master Oct 20, 2016
@bkeepers bkeepers deleted the parser branch October 20, 2016 04:38
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.

Figure out a better configuration syntax
1 participant