Andrew Lindesay [Canva]

Results 11 comments of Andrew Lindesay [Canva]

Hello @slott56 ; would it be possible to merge this PR in order to make `cel-python` compliant with the standard?

Please note that it *might* be necessary to augment this PR with a further adjustment to make `lark` >= 1.0 work. The change is in the file `celpy/celparser.py`. Additional kwargs...

## Before If I open the example found at `examples/python/simple_code_generator` from the [PR branch](https://github.com/bazelbuild/intellij/pull/6726) with the Intelli-J / Bazel plugin installed as the mainline build from `master` (from the Marketplace)...

Please see three related follow-up PRs; - [Adds a setting to control the additional `query`](https://github.com/bazelbuild/intellij/pull/6884/files) - [Documentation update](https://github.com/bazelbuild/intellij/pull/6886) - [Refactor query methods](https://github.com/bazelbuild/intellij/pull/6887) For those looking at this later, I added...

If this is merged, please disregard PRs [6886](https://github.com/bazelbuild/intellij/pull/6886) and [6887](https://github.com/bazelbuild/intellij/pull/6886). I note today that on this PR there are some test issues. ## Aspect Tests for IJ OSS Latest Stable...

@tpasternak ; I've been working on resolving problems (see earlier comment) with the testing harness out in relation to the `--aspects_parameters` used to convey the code generator Rules' names to...

This PR will now use `--action_env=...` instead of `--aspects_parameters=...` to convey the code generator Rule names to the aspect logic in order to get around problems related to needing to...

@tpasternak; I understand the problem with the action-env being available to all of the actions. I had considered this as a downside as well. As an alternative, would it be...

OK; leave it with me and I will see what I can do about making this a Bazel 7+ feature and not cause any breaking problems for earlier versions.