reports: Add Vamshi99's GSoC report

reports/vamshi99.md

@@ -0,0 +1,215 @@
+---
+layout: report
+report_year: '2018'
+permalink: /GSoC/2018/StatusReport/VamshiKrishna
+student: Vamshi Krishna Bommerla
+organisation: coala
+organisation_link : https://coala.io
+project: Implement GitMate automations and plugins for coala
+project_link: https://summerofcode.withgoogle.com/projects/#5344316227059712
+tarball: https://rawgit.com/Vamshi99/GSoCReport/master/patches.tar.gz
+sha256sum: 406c470fd6db57b6eeb6c834ce96983adb44235bf3f006124e7b94da039dd17f
+mentors: >
+ [Naveen Kumar Sangi](https://github.com/nkprince007),
+ [Lasse Schuirmann](https://github.com/sils),
+ [Manvendra Singh](https://github.com/manu-chroma),
+ [Hizkia Felix](https://github.com/HizkiFW)
+phase:
+ - Bonding : https://gitlab.com/coala/GSoC/GSoC-2018/milestones/4
+ - Phase 1 : https://gitlab.com/coala/GSoC/GSoC-2018/milestones/5
+ - Phase 2 : https://gitlab.com/coala/GSoC/GSoC-2018/milestones/6
+ - Phase 3 : https://gitlab.com/coala/GSoC/GSoC-2018/milestones/7
+bio: >
+ I am third year student of B.Tech Computer Science Engineering at Amrita
+ Vishwa Vidyapeetham, Amritapuri, Kerala. I participated in GSoC and worked
+ with [coala](https://coala.io) and [GitMate](https://gitmate.io) to implement
+ GitMate plugins and automation which help coala development workflow. I worked
+ on adding plugins which auto-reject, auto-review, auto-assign PRs and notify
+ merge conflicts in a PR. I also implemented a assign request plugin to assign
+ issues to a user if he/she satisfies defined eligibilty conditions.
+social:
+ - GitHub:
+   - username: Vamshi99
+   - link: https://github.com/Vamshi99
+ - GitLab:
+   - username: Vamshi99
+   - link: https://gitlab.com/Vamshi99
+ - Gitter:
+   - username: Vamshi99
+   - link: https://gitter.im/Vamshi99
+ - OpenHub:
+   - username: Vamshi99
+   - link: https://www.openhub.net/accounts/Vamshi99
+email: [email protected]
+blog: https://vamshi99.blogspot.com
+activity:
+ - 1:
+   - repo: cEPs
+   - link: https://github.com/coala/cEPs/commit/a4b2ab35243dc0b6aebc27986d33336d3bf80a64
+   - details: >
+      cEP-0024.md: Add cEP for GitMate Plugins
+ - 2:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/225a88e695794535f50996b4997466b72e9d47ea
+   - details: >
+      Add gitmate pr rejector plugin
+ - 3:
+   - repo: IGitt
+   - link: https://gitlab.com/gitmate/open-source/IGitt/commit/847afeb5460ce2866ecb2cc38593c0c486aa0271
+   - details: >
+      Commit.py: Add mentioned_issues, closes_issues propeties
+ - 4:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/05062b4f9758068291445f87469e4d9d2a4f9707
+   - details: >
+      Upgrade IGitt
+ - 5:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/3d2164d71722e55a6432a8911149c64897851564
+   - details: >
+      Add Fixes vs Closes automated checking plugin
+ - 6:
+   - repo: IGitt
+   - link: https://gitlab.com/gitmate/open-source/IGitt/commit/4a044453f5d34c9f1b8227b7f57b8a0fd0caba57
+   - details: >
+      MergeRequest.py: Add fix_issues, close_issues, resolve_issues properties
+ - 7:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/ffd554de617d619809f11b0284eaa032c2d9df83
+   - details: >
+      Auto assign issue based on commit mention in PR
+ - 8:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/0eec2ec32e7afd7d017cad4c67a741c688e1bfc4
+   - details: >
+      apps.py: Add helper function to fetch plugin settings
+ - 9:
+   - repo: IGitt
+   - link: https://gitlab.com/gitmate/open-source/IGitt/commit/e2224d7d1f61a34065f741535e53e676dc51ff4b
+   - details: >
+      Add Team class
+ - 10:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/f9d08c9a88c0b958a4c2ec5eb813fcd124b5300e
+   - details: >
+      Mark PR as WIP if coala plugin detects errors
+ - 11:
+   - repo: IGitt
+   - link: https://gitlab.com/gitmate/open-source/IGitt/commit/c4abdd4f9367af8ce73f39f525383def68de6b62
+   - details: >
+      MergeRequest: Add mergeable property
+ - 12:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/db76142d3ecb1e6cd9cd10885ae2475cb07c4d3d
+   - details: >
+      Notify merge conflicts in a PR
+ - 13:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/ad5480c9b0a124d98020a4ad3fd3caf20fa1e321
+   - details: >
+      pr_rejector: Support regex in branch names
+ - 14:
+   - repo: IGitt
+   - link: https://gitlab.com/gitmate/open-source/IGitt/commit/94735ff0f1024d6f293db90516087d8e4d2c353d
+   - details: >
+      Organization: Add issues property and filter_issues method
+ - 15:
+   - repo: IGitt
+   - link: https://gitlab.com/gitmate/open-source/IGitt/commit/fff0d568f1f2731c5055995ff19bbc93be842d5f
+   - details: >
+      Repository: Filter issues by label, assignee
+ - 16:
+   - repo: IGitt
+   - link: https://gitlab.com/gitmate/open-source/IGitt/commit/c025311167b07da5e825433a4d4d5e376adfed62
+   - details: >
+      GitLabUser: Fetch user id from username
+ - 17:
+   - repo: mr-rebaser
+   - link: https://gitlab.com/gitmate/open-source/mr-rebaser/commit/fad8305823a1058e93cd2397b5239a882b66c3f9
+   - details: >
+      Add squash pull request command
+ - 18:
+   - repo: mr-rebaser
+   - link: https://gitlab.com/gitmate/open-source/mr-rebaser/commit/3124c07ca5cd46acf985dc25fe22889e5a4744d0
+   - details: >
+      Dockerfile: Install requirements
+ - 19:
+   - repo: mr-rebaser
+   - link: https://gitlab.com/gitmate/open-source/mr-rebaser/commit/68360f3f0fac686907c7eb44c1a70bfa9e276d42
+   - details: >
+      run.py: Update squash command
+ - 20:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/dda98a8ab67d01376d3199bb82260838cebab7b0
+   - details: >
+      rebaser: Add squash command
+ - 21:
+   - repo: IGitt
+   - link: https://gitlab.com/gitmate/open-source/IGitt/commit/f4ec3c602745bcf08405479a684876cc1080a21f
+   - details: >
+      Issue: Add available_assignees property
+ - 22:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/a00c0265ff4a3da2eca0a1a96cb84ced033a5cd3
+   - details: >
+      Add assign and unassign request command
+ - 23:
+   - repo: gitmate-2
+   - link: https://gitlab.com/gitmate/open-source/gitmate-2/commit/4f14c3f1acfedb89489fd6645a05f14ac73a2b1d
+   - details: >
+      Update requirements.txt
+---
+
+### Implement GitMate automations and plugins for coala
+
+#### Work Done
+
+1. New Plugins added in GitMate:
+   1. Auto-assign issue based on commit mention in a Pull Request.
+   2. Notify merge conflicts in a Pull Request.
+   3. Auto review of Fixes vs closes keywords in a Pull request.
+   4. Auto reject pull requests opened on specified branches.
+   5. Squash commits in a Pull Request.
+   6. Request assignment to an issue.
+2. Add WIP label and remove pending review label if coala plugin finds defects
+   in a Pull Request.
+3. IGitt enhancements:
+   1. Filter issues by label, assignee in a repository.
+   2. Fetch user id from username in GitLabUser.
+   3. Added issues property and filter_issues method in Organization class.
+   4. Added fix_issues, close_issues, resolve_issues properties in
+      PullRequest.
+
+#### Challenges
+
+I had difficulties with implementing GitLab API in IGitt. GitHub has teams and
+GitLab API doesn't support teams, so I had to re-design assign request plugin
+to work without teams. Similarly, GitLab API doesn't have any webhook for pull
+request review event, so I couldn't implement a feature in GitMate
+which labels pull request as per review done by maintainer.
+
+#### Work to be done
+
+##### [Mark review labels as per the review](https://gitlab.com/gitmate/open-source/gitmate-2/issues/296)
+
+To implement `Mark review labels as per the review` feature in GitMate, a
+webhook event is needed which can detect a pull request review. In GitHub,
+there is a web hook event called [PullRequestReviewEvent](https://developer.github.com/v3/activity/events/types/#pullrequestreviewevent)
+to know if a PR is reviewed. In GitLab, there are no webhook
+[events](https://docs.gitlab.com/ee/api/events.html) that can be detected as a
+pull request review. If PullRequestEvent webhook needs to be added in IGitt,
+it needs to support GitLab implementation also. So, this feature couldn't be
+implemented as of now.
+
+##### [Merge all pull request review plugins](https://gitlab.com/gitmate/open-source/gitmate-2/issues/242)
+
+Initially, the idea was to merge all review plugins like ack, approver,
+auto label pending review or WIP. But, merging different plugins just because
+they clear review queue, was a bad idea. Consider code analysis plugin,
+which marks work in progress if it detects an error, but it doesn't make sense
+if it is merged with review plugin. So, I thought of declaring the review
+labels settings in a new plugin or introduce something like global settings
+which could be reused by any plugin if needed. There would be an option in
+required plugins if they want to enable handling review labels in a pull
+request. There was a delay in implementing this because I couldn't find a
+solution for sharing settings between different plugins.