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.

Sign up for GitHub

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

Jump to bottom

Drop .> as syntax for forward links. #982

Closed
vpetrovykh opened this issue Dec 17, 2019 · 0 comments · Fixed by #984
Closed

Drop .> as syntax for forward links. #982

vpetrovykh opened this issue Dec 17, 2019 · 0 comments · Fixed by #984
Assignees
@vpetrovykh
Labels
documentation edgeql high priority
Projects
Alpha 2

Comments

@vpetrovykh
Copy link
Member

The .> is perfectly equivalent to . in paths in all cases. This makes it redundant. The original reasons for keeping it boil down to symmetry with .< and nothing else. In practice, it doesn't appear all that useful or helpful, but it can create a little user confusion (e.g. "is there a difference between this and .?")

In the final balance of things it appears that this syntax should be dropped.
@vpetrovykh vpetrovykh self-assigned this Dec 17, 2019
@vpetrovykh vpetrovykh added this to To do in Alpha 2 via automation Dec 17, 2019
@vpetrovykh vpetrovykh mentioned this issue Dec 17, 2019
Merged
vpetrovykh added a commit that referenced this issue Dec 17, 2019
@vpetrovykh
edgeql: Drop .> path operator.
669cce4 
Drop the redundant `.>` path operator in favor of using only `.`.

Fixes #982.
Alpha 2 automation moved this from To do to Done Dec 17, 2019
vpetrovykh added a commit that referenced this issue Dec 17, 2019
@vpetrovykh
edgeql: Drop .> path operator.
7056974 
Drop the redundant `.>` path operator in favor of using only `.`.

Fixes #982.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vpetrovykh vpetrovykh

Labels
documentation edgeql high priority
Projects
No open projects
Alpha 2
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

edgeql: Drop .> path operator. edgedb/edgedb
1 participant
@vpetrovykh