Skip to content

Commit

Permalink
Whitespace changes as v4
Browse files Browse the repository at this point in the history
  • Loading branch information
fabio-looker committed Jan 3, 2025
1 parent 77a50d2 commit 2d9dabc
Show file tree
Hide file tree
Showing 11 changed files with 15 additions and 10 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ The manifest can provide the following declarations:
### Release Notes

- [v2](https://looker-open-source.github.io/look-at-me-sideways/release-notes/v2)
- [v3](https://looker-open-source.github.io/look-at-me-sideways/release-notes/v3)
- [v3+](https://looker-open-source.github.io/look-at-me-sideways/release-notes/v3)

### Privacy Policy

Expand Down
7 changes: 6 additions & 1 deletion docs/release-notes/v3.md
Original file line number Diff line number Diff line change
Expand Up @@ -101,4 +101,9 @@ Patches:

- v3.4.1 - Dependency updates in LAMS for jsonpath-plus. Dependency updates in Rule Sandbox for webpack and sub-dependencies.
- v3.4.2 - v3.4.0 had introduced a change in behavior at the intersection of conditional comments and whitespace rule W1. Since whitespace behavior for conditional comments had not been formally defined or desrcribed, this release defines the conditional comment whitespace handling going forward, primarily chosen to mirror the in-Looker IDE behavior when a block is commented via the Ctrl+/ shortcut.
- v3.4.3 - Reverts v3.4.2, leaving whitespace handling as-is. New whitespace handling will be introduced as a breaking change in v4.
- v3.4.3 - Reverts v3.4.2, leaving whitespace handling as-is. New whitespace handling will be introduced as a breaking change in v4.

# v4

v3.4.0 had introduced a change in behavior at the intersection of conditional comments and whitespace rule W1. Since whitespace behavior for conditional comments had not been formally defined or desrcribed, this release defines the conditional comment whitespace handling going forward, primarily chosen to mirror the in-Looker IDE behavior when a block is commented via the Ctrl+/ shortcut. Conditional comments will preserve the whitespace before the comment, consume the comment character and up to one space or tab character following the comment character and/or conditional comment token.

14 changes: 7 additions & 7 deletions npm-shrinkwrap.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
"fromentries": "^1.3.2",
"jsonpath-plus": "^10.2.0",
"liyad": "^0.2.4",
"lookml-parser": "^6.10.0",
"lookml-parser": "^6.11.0",
"minimist": "^1.2.6",
"require-from-string": "^2.0.2"
},
Expand Down

0 comments on commit 2d9dabc

Please sign in to comment.