Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/1014
@ottobackwards If it's a feature branch, the PR on Github doesn't
automatically close. It only happens for merges into a master. It's a bit
annoying, but I'm not sure there's a whole lot to do
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1014
This is in, not sure why it isn't closed
---
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/1014
Ok, this looks good. +1 to go to the feature branch
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1014
ok, I updated the feature base to track master, and pulled that into this.
I also added support for `:=` @cestella
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1014
I'll expand the assignment to include := in literal stellar, maybe that
will bump some review
---
Github user JonZeolla commented on the issue:
https://github.com/apache/metron/pull/1014
I wonder if it's good enough just to keep `:=` and `=` around, but not have
that affect the format of `++`, `+=`, etc.
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1014
Can I ask how we do ++, --, += etc with the := notation?
---
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/1014
Do we want to expand assignment to `:=` as part of this PR or make it a
follow-on in this feature branch? I'm ok with either way.
---
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/1014
>Ultimately, I'd like to see us move to supporting = as that is just way
more intuitive for most people. Maybe that means supporting both = and := for
some period of time?
In my opinion
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/1014
> I am not sure how this PR impacts all the places where Stellar is used
and assignment .. Or maybe that is left to a future PR?
Sorry, its a little hard to digest that old PR, but I
Github user nickwallen commented on the issue:
https://github.com/apache/metron/pull/1014
> Should assignment be = or :=?
Ultimately, I'd like to see us move to supporting `=` as that is just way
more intuitive for most people. Maybe that means supporting both `=` and `:=`
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/1014
Should assignment be `=` or `:=`? I'm not super opinionated other than we
have been using `:=` and we'd break existing installations if we move to `=`,
right?
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1014
ok, @cestella I have fixed up the feature branch and this pr so it is clean
---
Github user ottobackwards commented on the issue:
https://github.com/apache/metron/pull/1014
@cestella I rebased this on the new feature branch ( after rebasing on the
same master ) and I get all of these other commits. I don't know how to get
rid of them?
---
14 matches
Mail list logo