Skip to content
This repository has been archived by the owner on Oct 17, 2024. It is now read-only.

Add more operators #107

Merged
merged 7 commits into from
Apr 4, 2019
Merged

Add more operators #107

merged 7 commits into from
Apr 4, 2019

Conversation

yaziine
Copy link

@yaziine yaziine commented Apr 2, 2019

This PR adds the remaining operators of #103.

It fixes #101, #104, #105, #106.

@yaziine yaziine requested review from tealeg and asdine April 2, 2019 15:10
Copy link
Contributor

@tealeg tealeg left a comment

Choose a reason for hiding this comment

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

👍

rule/expr.go Outdated Show resolved Hide resolved
@yaziine
Copy link
Author

yaziine commented Apr 3, 2019

@asdine 🆙

@yaziine yaziine force-pushed the add-more-operators branch from 0863fd9 to 840da9a Compare April 3, 2019 09:55
@yaziine yaziine changed the base branch from add-gt-operator to release-v0.6 April 3, 2019 09:57
@asdine
Copy link
Contributor

asdine commented Apr 3, 2019

Waiting for this discussion to be done, I have the same issue in all the PRs #103 (comment)

@yaziine yaziine mentioned this pull request Apr 3, 2019
@yaziine yaziine force-pushed the add-more-operators branch from 840da9a to aa200a0 Compare April 3, 2019 14:30
@yaziine yaziine requested a review from tealeg April 3, 2019 16:03
@yaziine
Copy link
Author

yaziine commented Apr 3, 2019

@asdine @tealeg 🆙 🙏

@yaziine yaziine merged commit 6a178bc into release-v0.6 Apr 4, 2019
@yaziine yaziine deleted the add-more-operators branch April 4, 2019 13:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants