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

Unsupported multi-dimensional array indexing #490

Closed
extradosages opened this issue Oct 12, 2021 · 1 comment · Fixed by #504
Closed

Unsupported multi-dimensional array indexing #490

extradosages opened this issue Oct 12, 2021 · 1 comment · Fixed by #504
Labels
bug Something isn't working

Comments

@extradosages
Copy link

extradosages commented Oct 12, 2021

Seems like the new multi-dimensional array indexing syntax introduced in JuliaLang/julia#33697 and baked into Julia 1.7 is not supported.

Using JuliaFormatter v0.18.0 and Julia 1.7.0-rc1.

Diffs after linting, demonstrating the issue:

@@ -28,12 +28,12 @@ end
             [Pro("pro id", 10)],
             10,
             [Visit("visit id", [1 => 3])],
-            [1.5;;],
-            [0;;;],
-            [0.5;;],
-            [0;;],
-            [0;;;],
-            [true;;]
+            [1.5;],
+            [0;],
+            [0.5;],
+            [0;],
+            [0;],
+            [true;],
         )
 
         route = solve(spec)
@@ -49,12 +49,12 @@ end
                     [Pro("pro id", 10)],
                     10,
                     [Visit("visit id", [3 => 4])],
-                    [5;;],
-                    [0;;;],
-                    [1;;],
-                    [0;;],
-                    [0;;;],
-                    [true;;]
+                    [5;],
+                    [0;],
+                    [1;],
+                    [0;],
+                    [0;],
+                    [true;],
                 )
@domluna domluna added the bug Something isn't working label Oct 12, 2021
@domluna
Copy link
Owner

domluna commented Oct 12, 2021

not supported yet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants