Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [platform-dev] Github workflow

Wim,

Comments below.

On 22.03.2022 10:46, Wim Jongman wrote:
In BIRT, Windowbuilder, and Nebula we successfully use the following workflow:

  1. Fork the main repo
  2. Create an issue in the main repo e.g. "My Issue" #1 (#1 being the next ID)
  3. Make changes in a branch in your fork (not in master/main)!!!
  4. Commit with "My Issue #1"(include #1)

A few "concerns" here.  This approach does look good in the github views, but the issue URI is implicit.  That works well (on the github site) when the issue is open in the same repo as the commit, but I expect in the long term, there may well be commits for a single issue across multiple repos.  Also, EGit knows nothing about such implicit links so will not show such links in the UI.

It's also possible to include the full issue link like this such that even in EGit, you can see an issue link that you can follow to the github site:

This looks like this in the github pages:

Note how the long issue link is displayed as #2 here.  Also note that the #3 is a link to the PR while #2 is a link to the issue.  So here the implicit URI is different depending on the location of the #<fragment>. I'm not sure how EGit might create/display such links in the future...

So perhaps it's worth considering including the full link to the issue in the body of the commit message?

Just a thought...

  1. Go to GitHub in the main repo. GH offers to create a PR
  2. Use "My Issue #1" as the topic. The PR will be called "My Issue #1" #2
  3. Start Review
  4. Keep committing on your fork branch
  5. PR is accepted
  6. Go to your fork and "Fetch upstream" [1]

Yes, this is a PITA. :-(    It's hard to keep something like the following up-to-date if you also have to go off into github and manually fetch each fork (if you have one) before Select All and to a Pull.

I wondered if there were some way to make this easier with multiple configured remotes, but I don't understand that approach well enough

[1] Fetching upstream is currently a pain. I have started a discussion here. Please also comment or upvote:

Cheers,

Wim 

_______________________________________________
platform-dev mailing list
platform-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/platform-dev

Back to the top