Conversation
Co-authored-by: yaboyshades <128257328+yaboyshades@users.noreply.github.com>
Co-authored-by: yaboyshades <128257328+yaboyshades@users.noreply.github.com>
Co-authored-by: yaboyshades <128257328+yaboyshades@users.noreply.github.com>
…-bdbe-78d4240878a3 Fix multi-line continuation spacing issue in interactive interpreter
|
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
There was a problem hiding this comment.
Pull Request Overview
This pull request fixes a spacing issue in the Mangle interpreter's multi-line continuation feature and standardizes quote usage in the documentation configuration. The fix ensures proper spacing is added when concatenating continued lines to prevent parsing issues.
Key changes:
- Added spacing logic to handle multi-line continuation properly in the interpreter
- Added comprehensive tests to verify the continuation fix works correctly
- Standardized single quotes to double quotes in Sphinx configuration
Reviewed Changes
Copilot reviewed 4 out of 6 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| interpreter/interpreter.go | Added spacing logic for multi-line continuation to prevent parsing issues |
| interpreter/interpreter_test.go | Added test cases to verify multi-line continuation functionality |
| readthedocs/conf.py | Standardized quote usage from single to double quotes |
| CONTINUATION_FIX.md | Added documentation explaining the problem and solution |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| // Add appropriate spacing between lines to avoid parsing issues | ||
| if nextLine != "" && !strings.HasSuffix(clauseText, " ") && !strings.HasPrefix(nextLine, " ") { | ||
| clauseText = clauseText + " " + nextLine | ||
| } else { | ||
| clauseText = clauseText + nextLine | ||
| } |
There was a problem hiding this comment.
The closing brace on line 362 appears to be misplaced or extra. The if-else block is properly closed on line 361, making this brace potentially incorrect and likely causing a compilation error.
No description provided.