Skip to content
This repository has been archived by the owner on Sep 14, 2020. It is now read-only.

Fix bug #261, JSONPath should include the function name #262

Merged
merged 2 commits into from
Dec 13, 2019

Conversation

zakkg3
Copy link

@zakkg3 zakkg3 commented Dec 3, 2019

Signed-off-by: Nicolas Kowenski [email protected]

One-line summary

Issue : #261

Description

The JSONPath was missing the function name create_fn

@zincr
Copy link

zincr bot commented Dec 3, 2019

🤖 zincr found 0 problems , 0 warnings

✅ Large Commits
✅ Specification
✅ Dependency Licensing

@nolar nolar added the documentation Documentation improvements label Dec 3, 2019
@zakkg3
Copy link
Author

zakkg3 commented Dec 4, 2019

Let me know if I can help with the coverage/test failing check.

@nolar
Copy link
Contributor

nolar commented Dec 7, 2019

The CI checks fail due to mypy type checking issues due to a mypy's regression — I've described and kind-of-fixed this in #265.

@nolar
Copy link
Contributor

nolar commented Dec 12, 2019

@zakkg3 You can now update from master (or rebase) — the CI/CD builds-tests should be fixed.

@nolar nolar merged commit a472d0c into zalando-incubator:master Dec 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Documentation improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants