-
Notifications
You must be signed in to change notification settings - Fork 206
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
Version kotlinc plugins by major version to handle API changes. #1011
Open
restingbull
wants to merge
9
commits into
master
Choose a base branch
from
restingbull/998-version-plugins
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Commits on Aug 13, 2023
-
Configuration menu - View commit details
-
Copy full SHA for a5dcc72 - Browse repository at this point
Copy the full SHA a5dcc72View commit details -
Move kotlinc_version and ksp_version to versions
Allows reuse when defining kotlinc versions. Supporting multiple plugin apis will require supporting multiple kotlincs.
Configuration menu - View commit details
-
Copy full SHA for afb9629 - Browse repository at this point
Copy the full SHA afb9629View commit details -
Create KOTLINC_INDEX, major version to kotlinc metadata
Managing multiple kotlinc version increases toil. To simplify things, introduce a centralized list of the support versions of kotlinc. Success for this is predicated on the plugin api remaining stable on point releases, which is anecdotally true.
Configuration menu - View commit details
-
Copy full SHA for 1977ce4 - Browse repository at this point
Copy the full SHA 1977ce4View commit details -
Split plugin compilation by major kotlinc version.
This provides a separate plugin for each major kotlinc release. Still requires wiring into the toolchain system (follow up PR), but simplifies using older versions of kotlin with less effort.
Configuration menu - View commit details
-
Copy full SHA for c53234a - Browse repository at this point
Copy the full SHA c53234aView commit details -
Configuration menu - View commit details
-
Copy full SHA for d6ccef2 - Browse repository at this point
Copy the full SHA d6ccef2View commit details -
Current stardoc cannot seem to handle dynamic key in dictionary comprehensions. The simple solution is to separate the KOTLINC_INDEX from the compiler repository definition. Has the added benefit of keeping development logic from production logic.
Configuration menu - View commit details
-
Copy full SHA for 67734a7 - Browse repository at this point
Copy the full SHA 67734a7View commit details -
This was apparently skipped on previous passes.
Configuration menu - View commit details
-
Copy full SHA for a6a96ee - Browse repository at this point
Copy the full SHA a6a96eeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 20b6462 - Browse repository at this point
Copy the full SHA 20b6462View commit details -
Configuration menu - View commit details
-
Copy full SHA for d1691fe - Browse repository at this point
Copy the full SHA d1691feView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.