Skip to content

Use JSON string for dictionary example#15898

Merged
jsoriano merged 1 commit intoelastic:feature/python3from
jsoriano:fix-dicts-in-fields-docs
Jan 28, 2020
Merged

Use JSON string for dictionary example#15898
jsoriano merged 1 commit intoelastic:feature/python3from
jsoriano:fix-dicts-in-fields-docs

Conversation

@jsoriano
Copy link
Copy Markdown
Member

Some versions of python cannot guarantee order of dictionaries, so same
dictionary can generate different documentation strings, and this makes
make check to fail in some python 3 versions.

Use a JSON string so its representation doesn't depend on how python
encodes it.

Also, fix versions in python-env for root directory.

Some versions of python cannot guarantee order of dictionaries, so same
dictionary can generate different documentation strings. Use a JSON
string so its representation doesn't depend on how python encodes it.

Also, fix versions in python-env for root directory.
@jsoriano jsoriano requested a review from a team as a code owner January 28, 2020 15:07
@jsoriano jsoriano self-assigned this Jan 28, 2020
@jsoriano jsoriano requested review from andrewkroh and removed request for a team January 28, 2020 15:08
@jsoriano jsoriano mentioned this pull request Jan 28, 2020
63 tasks
@jsoriano
Copy link
Copy Markdown
Member Author

jenkins, test this please

@jsoriano jsoriano merged commit ae546c7 into elastic:feature/python3 Jan 28, 2020
@jsoriano jsoriano deleted the fix-dicts-in-fields-docs branch January 28, 2020 23:08
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Some versions of python cannot guarantee order of dictionaries, so same
dictionary can generate different documentation strings. Use a JSON
string so its representation doesn't depend on how python encodes it.

Also, fix versions in python-env for root directory.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants