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

Migrate to Adapt #72

Closed
davquar opened this issue Aug 27, 2021 · 0 comments · Fixed by #83
Closed

Migrate to Adapt #72

davquar opened this issue Aug 27, 2021 · 0 comments · Fixed by #83
Labels
enhancement New feature or request improvement Improvements to the code
Milestone

Comments

@davquar
Copy link
Owner

davquar commented Aug 27, 2021

The Adapt intent parser is more powerful and flexible; it also doesn't have the infamous bugs of Padatious.

To migrate it should be sufficient to:

  • convert the intent strings into regexes.
  • appropriately tune the intent builder with require and optionally to allow both long and compact intents.

As an alternative to the last point, it is also possible to use both parsers: Adapt for compact intents and Padatious for simple ones.

@davquar davquar added the improvement Improvements to the code label Aug 27, 2021
@davquar davquar changed the title Decide whether to migrate to Adapt Migrate to Adapt Aug 29, 2021
@davquar davquar added the enhancement New feature or request label Aug 29, 2021
@davquar davquar added this to the Iteration 2 milestone Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request improvement Improvements to the code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant