Topic: Move Widelands to GitHub
GunChleoc Topic Opener |
Posted at: 2019-09-15, 11:56
Yes, this is true.
We should definitely go over the documentation again once we have established our workflow. I have already added the instructions for the workflow that you just used. Busy indexing nil values Top Quote |
Nordfriese |
Posted at: 2019-09-15, 13:47
Will launchpad merge requests also be imported automatically or do I need to manually open pull requests for branches I had in review? Top Quote |
GunChleoc Topic Opener |
Posted at: 2019-09-15, 16:23
You will have to create new pull requests. If you still need to access the discussion on Launchpad, I have linked the branches in the first post of this thread, and you can still access the deactivated merge requests via their branches. Busy indexing nil values Top Quote |
stonerl |
Posted at: 2019-09-15, 16:49
@hessenfarmer if you use git just with GitHub. Give the official Github Desktop tool a try. Top Quote |
hessenfarmer |
Posted at: 2019-09-15, 19:59
ok I Think I am getting there. I did a test commit to my fork at the balancing branch. Top Quote |
GunChleoc Topic Opener |
Posted at: 2019-09-15, 20:09
Very glad to hear that you are figuring it out. Your strategy is good Please let us know if you need any more help, and if you have any ideas about improving the instructions in the wiki. AppVeyor and Travis are already up and running. I have configured them to run a build on Pull Requests, just like we used to have for Merge Requests. In addition to that, they will create a build each time somebody pushes to a branch in https://github.com/widelands/widelands Since I am migrating a lot of branches right now, we have a bit of a backlog, but things should be back to normal for Travis & AppVeyor after a week or so. I'm already pushing most of my branches to my fork rather than the official one to reduce the load on the builds. Travis has a separate tab for Pull requests: https://travis-ci.org/widelands/widelands/pull_requests And the branch builds are listed here: https://travis-ci.org/widelands/widelands/branches AppVeyor ist unchanged, with branches and pull requests on one page: https://ci.appveyor.com/project/widelands-dev/widelands/history Busy indexing nil values Top Quote |
stonerl |
Posted at: 2019-09-17, 11:37
How do we want to version builds from GitHub? The bzr based builds looked like this:
e.g.
How do we want to use this with git? Currently, it looks like this:
There are at least 3 possible schemes
This would look like this:
What do you think? Top Quote |
GunChleoc Topic Opener |
Posted at: 2019-09-17, 12:52
I'd go for 2 or 3. If we use the hash, keep the dash, because hashed can start with a letter. The revision looks like it's sequential, which is more intuitive - can it also be used to easily find the commit both in the GitHub UI and in the git client? Busy indexing nil values Top Quote |
stonerl |
Posted at: 2019-09-17, 13:35
You get this by using:
Another way could be, having both. This references the commit and a sequential number so it is easy to so how many commits versions differ from each other.
Top Quote |
GunChleoc Topic Opener |
Posted at: 2019-09-17, 13:49
Having both will make the string too long in the UI. Let's go with option 2 with the dash On another note, most of the issues in Widelands-media were also issues in Widelands itself, so I moved the few remaining ones to the Widelands bug tracker. They all carry the media label. Busy indexing nil values Top Quote |