Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AutoPR] TODO use jsonpath_ng instead #131

Open
github-actions bot opened this issue Nov 5, 2023 · 0 comments
Open

[AutoPR] TODO use jsonpath_ng instead #131

github-actions bot opened this issue Nov 5, 2023 · 0 comments
Labels

Comments

@github-actions
Copy link
Contributor

github-actions bot commented Nov 5, 2023

TODO use jsonpath_ng instead

class ContextDict(dict[ContextVarName, Any]):
def get_path(self, path: ContextVarPath) -> Any:
"""
Get a value from the context by path (e.g., `pull_request.title`).
"""
# TODO use jsonpath_ng instead
path_parts = path.split(".")
value = self
try:
for part in path_parts:
if isinstance(value, pydantic.BaseModel):

To resolve this TODO, you can start by replacing the existing code that uses jsonpath with the jsonpath_ng library. Make sure to update any necessary imports and modify the code to use the new library's syntax and functionality. Test the changes thoroughly to ensure they work as expected.

@github-actions github-actions bot added the medium label Nov 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

0 participants