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

Add jsonpath module #52891

Open
wants to merge 15 commits into
base: devel
from

Conversation

Projects
None yet
2 participants
@zhan9san
Copy link

zhan9san commented Feb 24, 2019

SUMMARY

Add a new module jsonpath. It is similar to xpath. But it just supports simple expressions.

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

jsonpath

ADDITIONAL INFORMATION

It can modify the value in json file via jsonpath syntax


@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Feb 24, 2019

The test ansible-test sanity --test pylint [explain] failed with 2 errors:

lib/ansible/modules/files/jsonpath.py:147:76: undefined-variable Undefined variable 'json_file'
lib/ansible/modules/files/jsonpath.py:156:53: trailing-whitespace Trailing whitespace

The test ansible-test sanity --test pep8 [explain] failed with 3 errors:

lib/ansible/modules/files/jsonpath.py:120:1: E302 expected 2 blank lines, found 1
lib/ansible/modules/files/jsonpath.py:156:54: W291 trailing whitespace
lib/ansible/modules/files/jsonpath.py:203:1: E305 expected 2 blank lines after class or function definition, found 1

click here for bot help

@ansibot ansibot removed the needs_ci label Feb 24, 2019

@ansibot ansibot removed the stale_ci label Feb 24, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

ansibot commented Feb 24, 2019

@EvanK @ahtik @bendoh @cmprescott @dhozac @jhoekx @jirutka @jpmens @krzysztof-magosa @luisperlaz @noseka1 @pileofrogs @ptux @ribbons @sfromm @sm4rk0 @tbielawa @tima @yaegashi

As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add shipit if you would like to see it merged.

click here for bot help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.