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

Replace protected with private access modifier for generated constructors with explicit Parcel parameter #63

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

mmBs
Copy link

@mmBs mmBs commented Feb 24, 2017

Hi Michał,

I replaced protected with private access modifier for generated constructors with explicit Parcel parameter to remove lint warning from the generated code.
Also, I changed a project structure and currently the META-INF folder with plugin.xml file is in resources folder (it is more up-to-date now).
I built and tested the plugin (I bumped version to 0.7.1) with the newest IntelliJ IDEA Community Edition (2016.3.4 Build #IC-163.12024.16) + JDK 1.8. It is fully operational.

Regarding the Android sample app, I bumped all Gradle dependencies to the newest and stable ones. All tests passed flawlessly.

I really appreciate your work. The plugin is really helpful and I hope you will find my updates useful.

Thanks,
Michal

mmBs added 5 commits February 24, 2017 17:02
for generated constructors with explicit Parcel parameter.

Also, group class imports are removed.
- Currently META-INF folder and plugin.xml are inside the resources folder.
- add new version --> 0.7.1
- add new changes
- Gradle Build Tools --> 2.2.3
- Compile Sdk Version --> 25
- Build Tools Version --> 25.0.2
- Target Sdk Version --> 25
- Increase Compatibility to Java Version 1.7
- Support Library --> 25.2.0
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