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

Fix on emulator-start goal #762

Closed
wants to merge 7 commits into from
Closed

Fix on emulator-start goal #762

wants to merge 7 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Aug 12, 2017

According the issue: #752,
I have fixed the goal.
Every informations about the fix are: https://github.com/amanganiello90/android-maven-plugin/blob/fixEmulatorStartBranch/noteEmulatorStartFix.txt

To contribute on this and other developments you can do a donation on : https://github.com/amanganiello90/maven-gitflow-plugin

Thanks you.

@@ -669,6 +669,7 @@
instead -->
<configLocation>src/conf/maven_checks.xml</configLocation>
<enableRulesSummary>false</enableRulesSummary>
<failOnViolation>false</failOnViolation>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not disable this check. Instead ensure that your code passes the style check.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know what format the maven check plugin wants and as doing it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The checkstyle errors will show the details and the codestyle is configured in the maven_checks.xml file. Your IDE should automatically pick it up and hightlight things as well. At least Eclipse does that.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My Eclipse IDE doesn't apply the format according the maven_checks.xml file.


1)SET ANDROID_HOME environment variable and in the path %ANDROID_HOME%/tools and platform-tools

2)User properties _android.devices_ or _android.device_ maybe are not supported because will run always the emulator.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is that?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because in my method to run android emulator I don't use the "AndroidDebugBridge" object that maybe return the devices set.

@mosabua
Copy link
Member

mosabua commented Aug 15, 2017

The file is completely revamped so I cant tell what changed. Please update the PR to limit the changes stick with the code style.

@ghost
Copy link
Author

ghost commented Aug 15, 2017

I have refactored the commit history in order to view only the changing done in the Emulator class.

@ghost
Copy link
Author

ghost commented Nov 14, 2017

@mosabua any problem to merge the pull request?

@ghost ghost closed this Nov 14, 2017
@ghost ghost reopened this Nov 14, 2017
@ghost
Copy link
Author

ghost commented Aug 22, 2018

@mosabua any news to merge this pull request?

@ghost ghost closed this by deleting the head repository Feb 7, 2024
This pull request was closed.
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