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

Library evolution #285

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

abel3cl
Copy link

@abel3cl abel3cl commented Feb 11, 2023

When building with flag -enable-library-evolution,
@inlinable init do not compile with the error:
Screenshot 2023-02-11 at 17 14 59

However marking @usableFromInline a public init is not allowed.

Screenshot 2023-02-11 at 17 15 31

This PR creates an internal init as suggested by the first error that gets called from the public init

Refs:

Adds build-for-library-evolution stage
Makes internal @usableFromInline inits and calls it from it's respective public @inlinable init
Adds library-evolution
@danthorpe
Copy link

danthorpe commented Feb 13, 2023

This is great @abel3cl, we've been looking to get this working too!

@stephencelis
Copy link
Member

@abel3cl Really sorry to have lost track of this. Any chance you're able to play these changes back on the latest so that we can check them out soon?

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.

3 participants