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

Fix wrong := REPL documentation #39975

Merged
merged 1 commit into from
Mar 10, 2021

Conversation

Liozou
Copy link
Member

@Liozou Liozou commented Mar 10, 2021

Fixes #39427

It seems := deserved special casing among the operators finishing with = since it is one of four unsupported assignment operators (, and :=), as written here:

julia/src/julia-syntax.scm

Lines 4431 to 4433 in b2d66c9

;; unsupported assignment operators
((≔ ⩴ ≕ :=)
(error (string "unsupported assignment operator \"" (deparse (car e)) "\"")))

Should they have special REPL documentation stating that they cannot be defined? With this PR, the default behavior is reinstated:

help?> :=
search:

  No documentation found.

  Binding := does not exist.

and the same for the other three, which I think should be enough since using any of these four operators yields:

julia> a := b
ERROR: syntax: unsupported assignment operator ":="

@Liozou Liozou added the docsystem The documentation building system label Mar 10, 2021
@simeonschaub
Copy link
Member

Should they have special REPL documentation stating that they cannot be defined? With this PR, the default behavior is reinstated:

Yes, that might make sense, but that can be another PR. The current behavior is definitely wrong, so let's go with this for now.

@simeonschaub
Copy link
Member

Linux32 failure is likely because your branch is not based on top of latest master

@simeonschaub simeonschaub merged commit abde2f1 into JuliaLang:master Mar 10, 2021
@simeonschaub simeonschaub added the backport 1.6 Change should be backported to release-1.6 label Apr 25, 2021
simeonschaub pushed a commit that referenced this pull request Apr 26, 2021
This PR fixes #40563 .
This is essentially a mirror of #39975 . Please refer to it for more information.
KristofferC pushed a commit that referenced this pull request May 4, 2021
KristofferC pushed a commit that referenced this pull request May 4, 2021
This PR fixes #40563 .
This is essentially a mirror of #39975 . Please refer to it for more information.

(cherry picked from commit 1a12495)
@KristofferC KristofferC mentioned this pull request May 4, 2021
45 tasks
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
This PR fixes JuliaLang#40563 .
This is essentially a mirror of JuliaLang#39975 . Please refer to it for more information.
jarlebring pushed a commit to jarlebring/julia that referenced this pull request May 4, 2021
This PR fixes JuliaLang#40563 .
This is essentially a mirror of JuliaLang#39975 . Please refer to it for more information.
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
This PR fixes JuliaLang#40563 .
This is essentially a mirror of JuliaLang#39975 . Please refer to it for more information.
johanmon pushed a commit to johanmon/julia that referenced this pull request Jul 5, 2021
This PR fixes JuliaLang#40563 .
This is essentially a mirror of JuliaLang#39975 . Please refer to it for more information.
@KristofferC KristofferC removed the backport 1.6 Change should be backported to release-1.6 label Jul 12, 2021
@Liozou Liozou deleted the wrongassignmentopdoc branch December 26, 2021 17:32
staticfloat pushed a commit that referenced this pull request Dec 23, 2022
staticfloat pushed a commit that referenced this pull request Dec 23, 2022
This PR fixes #40563 .
This is essentially a mirror of #39975 . Please refer to it for more information.

(cherry picked from commit 1a12495)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docsystem The documentation building system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

:= doc string seems wrong
3 participants