-
Notifications
You must be signed in to change notification settings - Fork 38
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 get the code #11
Fix get the code #11
Conversation
many more cross-links, and more detail in the menus
* make accordion so entire panel is link * put `test-both` first in the build help * remove obsolete submenu items in getting started
3ed0713
to
6a4559b
Compare
|
||
{% highlight bash %} | ||
hub fork; git submodule foreach 'hub fork' | ||
{% endhighlight %} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something I have just discovered worth noting! If you already have forks, this command still works - your fork is left alone but it still sets up the new remote.
Thanks @ahgittin, this improves usability(readability!) a lot. There's still some issues remaining, but IMO it's worth publishing these changes ASAP so I'm going to merge now and regenerate the main website. |
Some things to add after my experiences with submodules... I think that this command is necessary after the first git submodule foreach "git checkout master" Apache committers may benefit from these commands - it overrides the git config branch.master.pushRemote apache-git
git submodule foreach "git config branch.master.pushRemote apache-git" |
@richardcloudsoft I've applied and pushed the essential changes in fce745e . Regarding As for the other comments I could go either way. Probably slightly prefer it as it is, longwinded but thorough, but happy if someone cleans it up further. (OTOH removing the script to make it like this is was a great suggestion! Thanks.) |
Ok that's great, I think we're done for now. Thanks for taking the time to make the changes - I think it's made the first (developer) contact much more straightforward! |
Add LDAP_DOMAIN and fix brooklyn-* replacements
Address @rdowner 's comments on #1