Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [escet-dev] Contributing to ESCET
  • From: Dennis Hendriks <dh_tue@xxxxxxxxxxx>
  • Date: Sun, 11 Apr 2021 12:35:50 +0000
  • Accept-language: nl-NL, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=4PMfwB4J1q8dwLHhJDnkar2KZ/Ll3Rr0dzvKOJLOrdk=; b=D7Np2HUxB8c4Xd0qJ2zf16KUq/Tl5hcLTRsFoaNCPtYPD2oZRA8G6QkpYvxlHPb0eHaBVJovftvqUeo3BPAsuPTZ7zmqABKhVOmzeNMLqnETelcMXUACWLi2+OBMPi+VVYUQBSFOf+0cMw9WwuXFXSiSrexBP0lCVs4lqqKFCHc3unhmm5UrIYI0dqXHyn6azLxaBvNhnm5/ymTqGC4V82/PfBEon9M5g4EjT82yhMtJtlkFJ2XHgjvgVG4T9BrLawV6zC4rgGjgqm/WUa7s09HE2XByYzNlQEjg/u3GwsJurHWZPIV5wo4mmqfSbNYcw9LQbVWI/w1ic8SUxpB4Ww==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=AiPRUnNWbuO0skmSL54mL8ttmlv7iqMVmhObyZ2SujkMBmbvhqIhlSl/UqyofiTr6GwHS/Jhvek04MnACfD+slbSp54f1yOqPk913dCUEKVo1yEU8BFqEHJx7R/uYOgTf5M3dGAE6F+UTdyRevyS2dzn6XpG0QMHcgdYCordIVCqXy3KewjhJqD603a2WJMUmgWM/iMwxAkiPSd0rlD57Z9a8NHCBteVCl9a/3zwt+try2DnRncHLDu7xY+0NfuHf3jGZg+G15m5RyP/yFnKll5oSuZdKzTFcxJG4iiyeXTB5Ofc63rlzlGrNWc/OHQ/qrCTCTy/f+8vm34S1JpGrg==
  • Delivered-to: escet-dev@xxxxxxxxxxx
  • List-archive: <https://dev.eclipse.org/mailman/private/escet-dev/>
  • List-help: <mailto:escet-dev-request@eclipse.org?subject=help>
  • List-subscribe: <https://dev.eclipse.org/mailman/listinfo/escet-dev>, <mailto:escet-dev-request@eclipse.org?subject=subscribe>
  • List-unsubscribe: <https://dev.eclipse.org/mailman/options/escet-dev>, <mailto:escet-dev-request@eclipse.org?subject=unsubscribe>
  • Thread-index: AQHXLT7bV1MDUnW3+kaxByno5pV8UaqtjGUWgAARLICAADJVMIABQnsAgAAuWD8=
  • Thread-topic: [escet-dev] Contributing to ESCET

Hi Ferdie,

It indeed seems that admin rights are needed to use the auto mirroring feature.

However, you should be able to use the standard Git functionality to have multiple remotes. That is, typically you have a remote called 'origin' in your local Git repository to pull/push from/to your ESCET fork in GitLab. You can add a second remote for the official ESCET Git repo in GitLab. You can then also pull from there, and then push it to your ESET fork. This way, you can work with all 3 repos (local one, official one in GitLab, your fork in GitLab).

Both auto mirroring and the standard Git functionality are explained at https://forum.gitlab.com/t/refreshing-a-fork/32469/2. Maybe you can try out the standard Git functionality and report on it at https://gitlab.eclipse.org/eclipse/escet/escet/-/issues/27? It would help to know whether the information from the GitLab forum that I linked is sufficient for you.

Dennis


Van: escet-dev <escet-dev-bounces@xxxxxxxxxxx> namens Ferdie Reijnen <ferdie.reijnen@xxxxxxxxx>
Verzonden: zondag 11 april 2021 11:38
Aan: escet developer discussions <escet-dev@xxxxxxxxxxx>
Onderwerp: Re: [escet-dev] Contributing to ESCET
 
Hi Dennis,

When working on a second contribution, I tried to synchronize my forked repository with the original repository (I want to get !29 into the forked repository). However, I couldn't find a simple way to do that. I found this post: https://about.gitlab.com/blog/2016/12/01/how-to-keep-your-fork-up-to-date-with-its-origin/ However, the mirror setting is only available to GitLab administrators. 

I am now a bit lost. I don't think I am supposed to fork the repository each time I want to add a contribution?

It would be nice to add some instructions on the development page.

Kind regards,
Ferdie

On Sat, 10 Apr 2021 at 16:30, Dennis Hendriks <dh_tue@xxxxxxxxxxx> wrote:
Hi Ferdie,

The only suggestion I have is to include what you expect in the "signed-off-by" field, it is not mentioned in the handbook you link, I had to google it. 

This is actually no longer required. See https://gitlab.eclipse.org/eclipse/escet/escet/-/issues/50

For the issues on GitLab. I'm not sure if it is intended, but I was unable to add labels [...]

I don't know. I assume it is an access rights issue. We don't have access to change the configuration currently. This may change in the future, see https://gitlab.eclipse.org/eclipsefdn/gitlab/-/issues/17For now, we can just add the labels for you.

> [...] or assign myself to the issue. 

It seems we can only assign issues to committers. Maybe we can change that as well later. For now, just indicate in the comments that you're working on it.

Dennis


Van: escet-dev <escet-dev-bounces@xxxxxxxxxxx> namens Ferdie Reijnen <ferdie.reijnen@xxxxxxxxx>
Verzonden: zaterdag 10 april 2021 13:24
Aan: escet developer discussions <escet-dev@xxxxxxxxxxx>
Onderwerp: Re: [escet-dev] Contributing to ESCET
 
Hi Dennis,

Thanks for your quick response and for improving my contribution!

The guidelines are clear and I could easily follow them. I was not familiar with forking and then merging from the forked repository, but it was explained well. The only suggestion I have is to include what you expect in the "signed-off-by" field, it is not mentioned in the handbook you link, I had to google it. 

For the issues on GitLab. I'm not sure if it is intended, but I was unable to add labels or assign myself to the issue. 

Kind regards,
Ferdie

On Sat, 10 Apr 2021 at 12:31, Dennis Hendriks <dh_tue@xxxxxxxxxxx> wrote:
Hi Ferdie,

Thanks for trying out the contribution guidelines. If you have any suggestions how to improve the guidelines, or insights into what works well or can be improved, please let us know. It's good to see you managed to create the issue and merge request.

I used this very first ever non-committer contribution to test out the process of extending your merge request. I found out how to get the merge request into my local Git, and add commits to it. It turns out this leads to a new branch 'merge-requets/28', as it extends merge request !28 that you created. I then added a few commits, as part of reviewing your merge request. These don't appear in merge request !28 as that is from your branch in your own fork of the 'escet' repo on the Eclipse Foundation GitLab. Instead I had to create my own merge request !29 instead. I'll close your merge request !28. Please take a look at merge request !29 to see whether you think my changes are OK.

It is a bit confusing that merge request !29 is now based on branch 'merge-request/28' for issue #25. We may have to find a better way for that.

Thank you for contribution!

Dennis


Van: escet-dev <escet-dev-bounces@xxxxxxxxxxx> namens Ferdie Reijnen <ferdie.reijnen@xxxxxxxxx>
Verzonden: vrijdag 9 april 2021 14:50
Aan: escet-dev@xxxxxxxxxxx <escet-dev@xxxxxxxxxxx>
Onderwerp: [escet-dev] Contributing to ESCET
 
Hello everyone,

I contacted Albert to discuss the possibility of contributing some bug fixes to the ESCET project. He suggested to try to submit a contribution following the developer guidelines. I tried that with a small bug fix for a cif2cif transformation. There should be a merge request for that submission now.

If I did anything wrong or my submission is incomplete please let me know.

Kind regards,
Ferdie
_______________________________________________
escet-dev mailing list
escet-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/escet-dev
_______________________________________________
escet-dev mailing list
escet-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/escet-dev

Back to the top