-
Notifications
You must be signed in to change notification settings - Fork 14
Help with subtrees #77
Comments
I've also started the process like 2 weeks ago, and messed up with the steps so I've stopped to not mess with anything. I think we should wait for @snostorm to see what he would do with our branchs precisely, that would help to understand the steps. And indeed these commands are to be executed on a |
yeah the PR bit makes sense to get our changes in, but then what are the benefits of using subtrees? still confused :) |
I'm still a little confused too 😄 but I thought it was essentially to have a 'persistent' link between the two. And be able to modify ours to then push to theirs. |
Hey, I'm formulating a better answer and digging in to the two branches in question. Initial response:
In time, if we got enough i18n subtrees established, we could likely automate some of the importing to avoid the second PR. |
For the sake of speed I can also just do the subtree steps for now and work to help document them better after? |
I'm ok with this @snostorm. |
Hey,
@snostorm @tdd @yoannmoinet I'm calling the subtrees experts
I'm struggling with the subtrees thing we're trying to introduce. We've merged a couple of PRs recently into the
website
branch and I don't understand how to notify theiojs.org
repository so that it gets the latest updates.Here are the last two pull requests that we've merged into the
website
branch:#73
#74
I noticed that we have a
website
branch that isn't listed on the list of branches on thisiojs-fr
repository, and aiojs-fr_website_subtree
branch that I can't checkout locally. Why is that? Is the ìojs-fr_website_subtree` branch designed to receive the latest changes or was created for something else?Also, it seems that the commands that @snostorm mentioned should be executed on
iojs.org
for which we don't have write permissions:git fetch iojs-fr website git pull --squash -s subtree iojs-fr website git commit -m "Updates iojs-fr content from iojs/iojs-fr#website"
What am I missing?
Thanks,
Olivier
The text was updated successfully, but these errors were encountered: