cloud-custodian / cloud-custodian/cel-python
Cleanup use of `lark.Tree` vs. `Expression`
Open
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