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

Use JSON string for dictionary example #15898

Merged
merged 1 commit into from
Jan 28, 2020

Conversation

jsoriano
Copy link
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
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