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

The pipeline operator is now '|>'. #751

Merged
merged 3 commits into from Jan 8, 2013
Merged

The pipeline operator is now '|>'. #751

merged 3 commits into from Jan 8, 2013

Conversation

alexrp
Copy link

@alexrp alexrp commented Jan 7, 2013

Fixes #718.

@yrashk
Copy link
Contributor

yrashk commented Jan 7, 2013

Nice! It does seem to miss a /> operator deprecation clause, though.

@alexrp
Copy link
Author

alexrp commented Jan 7, 2013

Should /> go through proper deprecation, i.e. still be semi-supported in compiler/kernel but be marked deprecated, or just be renamed right away (as done here) with a change log entry?

@yrashk
Copy link
Contributor

yrashk commented Jan 7, 2013

My opinion is that it should go through a proper deprecation cycle and be
removed in 0.9. Until then it should just raise a warning. @josevalim ?

On Mon, Jan 7, 2013 at 3:42 PM, Alex Rønne Petersen <
notifications@github.com> wrote:

Should /> go through proper deprecation, i.e. still be semi-supported in
compiler/kernel but be marked deprecated, or just be renamed right away (as
done here) with a change log entry?


Reply to this email directly or view it on GitHubhttps://github.com//pull/751#issuecomment-11977853.

alexrp added 2 commits January 8, 2013 01:12
This commit can simply be reverted when the operator is to be removed.
@alexrp
Copy link
Author

alexrp commented Jan 8, 2013

Done.

josevalim pushed a commit that referenced this pull request Jan 8, 2013
The pipeline operator is now '|>'.
@josevalim josevalim merged commit caf2d82 into elixir-lang:master Jan 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Deprecate /> in favor of |>
3 participants