Allowing to specify patterns for ignored repos during org linking

[dennisoelkers]

This change adds a field "excludePattern" to the org model, which allows to specify a comma-separated list of repository names or substrings which are matched against the repository of an incoming or rechecked PR. If the string matches against the repo name, the PR is ignored and no
CLA check is done.

Fixes #157.

[CLAassistant]

All committers have signed the CLA.

[coveralls]

Coverage decreased (-0.4%) to 91.271% when pulling a965e13 on Graylog2:exclude-patterns-for-org-linking into e51b6fe on cla-assistant:master.

[KharitonOff]

Great job! 👍 Thank you for your contribution! 😄 It works good so far.
We appreciate it very much.

The only point I have: there is no way to find out which repos/pattern I've excluded after the org is linked. I could imagine to have this info as a popover placed on the org name or even better as a modal view, where the user can edit this entry.

[dennisoelkers]

Thanks for your quick review and feedback, I appreciate it very much! I am right now to add some test coverage for the isRepoExcluded function, so coveralls is happy again. After that I will add a popover to the organization list, that's a good idea!

I actually had in mind to add a function to the organization action menu which allows you to edit the pattern string, but that's a bit more work than the part which allows you to specify one during linking.

[coveralls]

Coverage increased (+0.07%) to 91.713% when pulling e9af3d1 on Graylog2:exclude-patterns-for-org-linking into e51b6fe on cla-assistant:master.

[coveralls]

Coverage increased (+0.07%) to 91.713% when pulling fbca91b on Graylog2:exclude-patterns-for-org-linking into e51b6fe on cla-assistant:master.

[dennisoelkers]

Is there a chance to get this merged and deployed to cla-assistant.io? We would have to set up our own instance otherwise.

[KharitonOff]

Sure. This week or next week I'm going to take some time for review. After that we can take over your changes into CLA assistant

[coveralls]

Coverage increased (+0.07%) to 91.713% when pulling 3ef8c3d on Graylog2:exclude-patterns-for-org-linking into e51b6fe on cla-assistant:master.

[coveralls]

Coverage increased (+0.07%) to 91.713% when pulling 23508bc on Graylog2:exclude-patterns-for-org-linking into e51b6fe on cla-assistant:master.

[KharitonOff]

Great job! Thank you for your contribution 👍
It looks good for me.
I did some minor changes on the UI: place the tooltip only on the org name and adjusted it's text for the orgs without exclude pattern. I hope you are ok with it.

[fr34kyn01535]

I would love to be able to edit the pattern once i linked a organization

[KharitonOff]

@fr34kyn01535 this feature is not implemented yet, but you are very welcome to make a contribution to this project 😏 See issue #182