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

ticket-3475 #3549

Closed
wants to merge 1 commit into from
Closed

Conversation

fkardashov
Copy link

this will avoid randomization in the output file

this will avoid randomization in the output file
@joschi
Copy link
Contributor

joschi commented Jun 3, 2020

Refs #3475
Refs #2828

This PR looks similar to #3476.
While it reduces the entropy in the generated specification, it doesn't cover all cases.
For example elements in a list (such as enums) aren't sorted and will still depend on various other factors.

@hiddewie
Copy link
Contributor

hiddewie commented Jul 1, 2020

Also ref:

It would be very good to at least start with deterministic output somewhere, like this PR. Enums and other 'listed' items can come later. This PR can also be extended for enums and other 'listed' items in the specification.

@frantuma
Copy link
Member

frantuma commented Dec 3, 2020

A solution has been implemented in #3740, see PR comment for details and usage scenarios. Closing this ticket, any further issues in this area should be reported in a new or not closed ticket.

@frantuma frantuma closed this Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants