cloud-custodian / cloud-custodian/cel-python

Cleanup use of `lark.Tree` vs. `Expression`

Open
#125 0 comments 0 reactions 0 assignees View on GitHub
enhancement good first issue
Dominant language
Python
Stars
174
Forks
40
PR merge metrics
No merged PRs in 30d

Description

Pick one and use it consistently. See `celpy/__init__.py` for the definition of the `Expression` type. If this is to be used throughout, move it to `celparser`,

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the Expression definition in celpy/__init__.py and locating uses of both lark.Tree and Expression. Determine whether Expression should move to celparser, then make the representation consistent throughout the affected code. Done means the project uses one chosen type consistently.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
compilers
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.