knallimall.org knallimall.org

User Tools

Site Tools

ochorocho_contribute_to_gitlab

Differences

This shows you the differences between two versions of the page.

Link to this comparison view

Both sides previous revisionPrevious revision
Next revision		Previous revision
ochorocho_contribute_to_gitlab [2020/05/23 13:13] – adminochorocho_contribute_to_gitlab [2020/07/03 09:23] (current) – [How i started] admin
Line 11: Line 11:
 This is what i came up with ... This is what i came up with ...
  
-As simple GUI to view available composer packages on a single page with the option to enbale/disable it for the Composer Registry +As simple GUI to view available composer packages on a single page with the option to enable/disable it for the Composer Registry 
  
 {{ ::first-shot-gitlab-composer-packages.png?1000 |}} {{ ::first-shot-gitlab-composer-packages.png?1000 |}}
Line 25: Line 25:
 At the beginning i was not aware of all the tools Gitlab offers for development and to maintain code quality. Starting with [[https://gitlab.com/gitlab-org/gitlab-development-kit/-/blob/master/doc/set-up-gdk.md|GDK]] which allowed me to get started very quickly and Rubocop to make sure code quality is as expected. At the beginning i was not aware of all the tools Gitlab offers for development and to maintain code quality. Starting with [[https://gitlab.com/gitlab-org/gitlab-development-kit/-/blob/master/doc/set-up-gdk.md|GDK]] which allowed me to get started very quickly and Rubocop to make sure code quality is as expected.
  
-In general, Gitlab will scan each and every corner of your code and will tell you what you did potentially wrong. +In general, Gitlab will scan each and every corner of your code and will tell you what you did potentially wrong. "Potentially wrong" ?! Well, in some cases you have to ignore Rubocop checks. Best is to talk to your reviewer if one exists.
- +
-"Potentially wrong" ?! Well, in some cases you have to ignore Rubocop checks. Best is to talk to your reviewer if one exists. +
  
 +Pipelines are amazing! Look at the jobs it will tell you all you need to know to write code for Gitlab.
 ===== Feedback kicks in or the stage where things go totally wrong ===== ===== Feedback kicks in or the stage where things go totally wrong =====
  
Line 49: Line 47:
 This ended up in 3 Merge Requests: This ended up in 3 Merge Requests:
  
-  * [[https://gitlab.com/gitlab-org/gitlab/-/merge_requests/17417|Add composer packages feature]] +  * [[https://gitlab.com/gitlab-org/gitlab/-/merge_requests/17417|Add composer packages feature]] - obsolete due to rewrite 
-  * [[https://gitlab.com/gitlab-org/gitlab/-/merge_requests/9306|Add GUI changes to show composer repository paths and copy button]]+  * [[https://gitlab.com/gitlab-org/gitlab/-/merge_requests/9306|Add GUI changes to show composer repository paths and copy button]] - Closed due to move to VueJS after a couple of month
   * [[https://gitlab.com/gitlab-org/gitlab/-/merge_requests/18054|Add composer packages docs]]   * [[https://gitlab.com/gitlab-org/gitlab/-/merge_requests/18054|Add composer packages docs]]
  
Line 80: Line 78:
 I thought i was cool at that point. Even though it was not working like packagist.com. But that was never my intention. I thought i was cool at that point. Even though it was not working like packagist.com. But that was never my intention.
  
-In the end GG was not happy about the solution building a package using the runner when the required files are already in Gitlab via the “Download Code/Tag” feature. So we agreed on starting from scratch and leave out the runner part. +In the end GG was not happy about the solution building a package using the runner when the required files are already in Gitlab via the “Download Code/Tag” feature. So we agreed on **starting from scratch** and leave out the runner part. 
  
 My initial idea was correct, but at that time there was no one paying attention to that detail. My initial idea was correct, but at that time there was no one paying attention to that detail.
Line 89: Line 87:
 ===== Luckily ===== ===== Luckily =====
  
-In feb 2020 GG started to write proper feature issues to make sure things are done right. While i was busy living life GG rewrote the entire thing and it will hopefully be available in [[https://gitlab.com/groups/gitlab-org/-/milestones/47|Gitlab 13.1 +In feb 2020 GG started to write proper feature issues to make sure things are done right. While i was busy living life GG rewrote the entire thing and it will hopefully be available in [[https://gitlab.com/groups/gitlab-org/-/milestones/47|Gitlab 13.1]] 
-]]+ 
 +Thanks to Mayra, Ray and of course GG for getting this out the door.
  
  
 ===== Summary ===== ===== Summary =====
  
-  - Read the docs ... read it again+  - Read the [[https://docs.gitlab.com/ce/development/|docs]] ... read it again
   - Make sure people understand what feature you are developing and why you did it this way   - Make sure people understand what feature you are developing and why you did it this way
   - Do not expect that reviewers are experts in that field you are developing a feature for. Make them experts by providing docs and detailed answers to their questions   - Do not expect that reviewers are experts in that field you are developing a feature for. Make them experts by providing docs and detailed answers to their questions
ochorocho_contribute_to_gitlab.1590239624.txt.gz · Last modified: 2020/05/23 13:13 by admin

Page Tools