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

Submodule's dir named after artifactId #16

Merged
merged 7 commits into from Aug 16, 2011

Conversation

manandbytes
Copy link
Contributor

Hi there! One more fix, this time for #7 (partially, for android-release only). To make it easier to review, I've made @c3c22d08 as a separate commit because it fixes an indentation only. It may be even thrown away at all.

Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
@cescoffier
Copy link
Contributor

Hi,

This looks pretty good. Could you do the same for the integration-test archetype ? I will merge into the master branch (and probably release it afterwards).

@manandbytes
Copy link
Contributor Author

Yup, I'll be back in couple days.

- explicitly defines modules and remove 'application*' from
  directories' name
- reference root artifact id via property
- move resources from application* to __rootArtifactId__*

Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
@manandbytes
Copy link
Contributor Author

Actually, these changes deprecate an android-archetypes-it module. It may be removed at all but I'm not sure if it make sense to move asserts from QuickstartArchetypeTest, ReleaseArchetypeTest and WithTestsArchetypeTest into a post-build hook script, http://maven.apache.org/plugins/maven-invoker-plugin/examples/post-build-script.html

@cescoffier
Copy link
Contributor

Hi,

Why does it 'deprecate' the integration tests ? They are run after the archetypes themselves, to should not be an issue. We just have to update the tests to match the new structure. no ?

@manandbytes
Copy link
Contributor Author

Sorry, my previous comment really belongs to #18.

@cescoffier cescoffier merged commit 2e3c062 into techatspree:master Aug 16, 2011
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.

None yet

2 participants