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

Sort constructors deterministically #3089

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vmishenev
Copy link
Member

  • Should keep the same order of constructors in K1 and K2?
  • Should a primary constructor be first?
  • Does it make sense to move sortDivergentElementsDeterministically to another stage? e.g. to the Documentable model. The current implementation ignores [SameMethodNamePageMergerStrategy].
    Although the compiler keeps the order of all declarations, the problem is actually for our extensions that are collected by Dokka. see Non deterministic sorting of merged extensions/properties #2784

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.

1 participant