-
1. Re: AeroGear github repo cleanup and setup
jbalunas Jan 11, 2012 11:59 AM (in response to jbalunas)So our initial structure might be something like:
/examples
/archetypes
/code-style
/parent
BTW - one issue with git submodules is it is likely that only repo's can be pointed too. That means AeroGear quickstarts and archetypes for jboss-as would each need their own repo. This is not ideal. Perhaps go with Wesley's suggestion and just have a readme fine in the AeroGear quickstarts/archetypes in jboss-as that points to our repo for the latest versions.
I'll need to talk with Pete about this, and also about possible workflows with this.
-
2. Re: AeroGear github repo cleanup and setup
jbalunas Jan 12, 2012 10:40 AM (in response to jbalunas)I had a quick talk with Pete, and he believe subtree merges might be a better approach than sub-modules, but he also warned that both approaches can make for more complex workflows and ?history?.
I'll follow up with Pete, as he said once we know the requirements we can pretty much make git do anything.
-Jay
-
3. Re: AeroGear github repo cleanup and setup
wesleyhales Jan 13, 2012 9:24 AM (in response to jbalunas)Why can't we just detach from the AS7 quickstarts?
-
4. Re: AeroGear github repo cleanup and setup
jbalunas Jan 13, 2012 10:22 AM (in response to wesleyhales)Because it will be important that our examples are included with the quickstarts. The primary reason being exposure for AeroGear, and vs versa for JBoss AS quickstarts.
So maybe with that it just makes sense to keep the complete /archetypes, and just keep whatever jboss-as quickstart/archetype we have in there? Then any other examples - like aerogear-7 - should exist on their own in aerogear repos.
-
5. Re: AeroGear github repo cleanup and setup
jbalunas Jan 17, 2012 10:14 AM (in response to jbalunas)Some notes from my talk with Pete.
- /aerogear/quickstarts[archetypes] repo
- Will only have aerogear related content
- And only content we want in JBoss-as q&a (quickstarts, archetypes)
- Must follow in sync with jboss-as coding standards
- Pete is investigating sub-tree's to accomplish this
- We need to get our repo's (q & a) into the shape we want them to integration.
- Repo name in aerogear might be as-quickstarts and as-archertypes or something like that
- The jboss-as team will only pull from a tag - M1, M2, etc...
- The jboss-as team must be notified of pull requests to review as well
- We'll work with Pete for initial setup
- likely M1
- then we'll work through instructions pete writes for others
- Will only have aerogear related content
- /aerogear/examples repo
- Will have non jboss-as quickstart examples for various things
- Any larger examples we want broken out will have their own
- /aerogear/denizen
- Example of a large example with its own repo
- /aerogear/code-styles
- Will fork jboss-as's coding standards
- /aerogear/parent
- Not sure we need this one - will think on it
- /aerogear/quickstarts[archetypes] repo
-
6. Re: AeroGear github repo cleanup and setup
jbalunas Jan 17, 2012 10:19 AM (in response to jbalunas)We also discussed git workflow and some rules for it.
We will need to review JBoss AS workflow, and talk with Jason to see if they have it written down. It is not following gitflow, but does have a multi-step workflow for commits, merges, rebasing, etc... It is similar to the dictator+trusted lieutenants approach from pro-git, but might have changes - http://progit.org/book/ch5-1.html
Rules:
- Never push your own commits upstream
- Always use a pull request, and have someone else merge it
- Don't use github pull request merge
- Merge/rebase yourself then close the pull request
- Never do any work in your master
- Always use topic branches
- When ever you merge rebase.
We need to get worked out quickly.
- Never push your own commits upstream
-
7. Re: AeroGear github repo cleanup and setup
wesleyhales Jan 17, 2012 10:59 AM (in response to jbalunas)Here's the AS7 workflow https://community.jboss.org/wiki/HackingOnAS7
-
8. Re: AeroGear github repo cleanup and setup
pmuir Jan 18, 2012 7:05 AM (in response to wesleyhales)Would recommend following the As7 workflow.
-
9. Re: AeroGear github repo cleanup and setup
jbalunas Jan 19, 2012 9:55 AM (in response to jbalunas)Create https://issues.jboss.org/browse/AEROGEAR-64 to cover the as-quickstart, and as-archetype setup.