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

Unpickle "privateWith", aka "package private" #583

Merged
merged 1 commit into from
Jan 18, 2021

Conversation

dwijnand
Copy link
Collaborator

Anything which is package private is now excluded from the problem
analysis MiMa does. This includes:

  • classes/traits/objects,
  • methods/fields,
  • method overloads,
  • as well as classes becoming private/public.

Fixes #53

@SethTisue
Copy link
Collaborator

which Scala versions does this work for? (2.11, 2.12, 2.13, 3?)

@dwijnand
Copy link
Collaborator Author

2.11-2.13. There's no pickle for Scala 3 (the new pickle is tasty, yum).

@SethTisue
Copy link
Collaborator

would "add TASTy support" be a good "help wanted" ticket on this, then?

@dwijnand
Copy link
Collaborator Author

I think EPFL is testing the waters there, or maybe just have intent to (I think at the Scala Center but not sure).

I'm not against it being another project - I'm not sure if if it's a better or worst thing if it's the same project. WDYT?

@SethTisue
Copy link
Collaborator

I don't see any reason it would or should be a separate project, but perhaps I'm missing something.

@dwijnand
Copy link
Collaborator Author

@sjrd am I aware correctly? Is someone actively or soon working on something in this area?

@sjrd
Copy link
Contributor

sjrd commented Nov 30, 2020

We're working on a project that will eventually allow us to build TASTy-MiMa. But TASTy-MiMa is anyway going to be complementary to MiMa, not replacing it. It will check backward TASTy compatibility, but that does not imply (nor is implied by) backward binary compatibility, which MiMa tests for.

@dwijnand dwijnand marked this pull request as draft December 8, 2020 13:47
@dwijnand dwijnand force-pushed the package-private branch 2 times, most recently from f2a0de6 to bd602ba Compare December 23, 2020 21:47
Anything which is package private is now excluded from the problem
analysis MiMa does.  This includes:

* classes/traits/objects,
* methods/fields,
* method overloads,
* as well as classes becoming private/public.
* case classes, or more generally class/object companions
* nested classes/objects (tested up to 3 levels)
@dwijnand dwijnand marked this pull request as ready for review December 24, 2020 01:47
@dwijnand
Copy link
Collaborator Author

dwijnand commented Dec 24, 2020

TODO: Make unpickling opt-out-able.

@dwijnand
Copy link
Collaborator Author

Nvm. Making it opt-out-able is a PITA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Allow ignoring package private classes
4 participants