Summary:

Add Matrix option to Integration

Review Request #11195 — Created Sept. 21, 2020 and discarded Oct. 14, 2020, 12:03 p.m.

Information
Owner:	ruonanjia
Repository:	rbintegrations
Branch:	master
Bugs:
Depends On:
Reviewers
Groups:	rbintegrations, students
People:

Description

Admins can choose to add Matrix Integration. This will send notifications to the registered matrix room about new review requets and updates to existing ones.

Testing Done

Tested in RB4.
Edited Slack unit test cases to test Matrix integration. All tests have passed except 2 of then complaining about missing image attachment
tests with errors are:
- test_notify_new_review_request_with_image_file_attachment
- test_notify_updated_review_request_with_new_image_attachments

Commits

Summary	Author
add Matrix integration option	ruonan

Screenshots

Issues

Description	From	Last Updated
F401 'json' imported but unused	reviewbot	Oct. 14, 2020, 10:05 a.m.
F401 'django.utils.six.moves.urllib.request.Request' imported but unused	reviewbot	Oct. 14, 2020, 10:06 a.m.
F401 'django.utils.six.moves.urllib.request.urlopen' imported but unused	reviewbot	Oct. 14, 2020, 10:07 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:07 a.m.
E303 too many blank lines (3)	reviewbot	Oct. 14, 2020, 10:07 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:07 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:07 a.m.
E303 too many blank lines (2)	reviewbot	Oct. 14, 2020, 10:07 a.m.
F401 'django.utils.six.moves.urllib.request.Request' imported but unused	reviewbot	Oct. 14, 2020, 10:07 a.m.
F401 'django.utils.six.moves.urllib.request.urlopen' imported but unused	reviewbot	Oct. 14, 2020, 10:07 a.m.
F841 local variable 'response' is assigned to but never used	reviewbot	Oct. 14, 2020, 10:07 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 14, 2020, 10:07 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:07 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:07 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:09 a.m.
E501 line too long (86 > 79 characters)	reviewbot	Oct. 14, 2020, 10:09 a.m.
E501 line too long (94 > 79 characters)	reviewbot	Oct. 14, 2020, 10:10 a.m.
E501 line too long (82 > 79 characters)	reviewbot	Oct. 14, 2020, 10:11 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:11 a.m.
F401 'matrix_client.api.MatrixHttpApi' imported but unused	reviewbot	Oct. 14, 2020, 10:12 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:12 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:13 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:13 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:13 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:14 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:14 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:15 a.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:29 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E501 line too long (121 > 79 characters)	reviewbot	Oct. 14, 2020, 10:16 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:16 a.m.
E302 expected 2 blank lines, found 1	reviewbot	Oct. 14, 2020, 10:16 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:17 a.m.
W293 blank line contains whitespace	reviewbot	Oct. 14, 2020, 10:17 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:18 a.m.
E501 line too long (103 > 79 characters)	reviewbot	Oct. 14, 2020, 10:21 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:26 a.m.
F821 undefined name 'self'	reviewbot	Oct. 14, 2020, 10:31 a.m.
E126 continuation line over-indented for hanging indent	reviewbot	Oct. 14, 2020, 10:31 a.m.
F821 undefined name 'json'	reviewbot	Oct. 14, 2020, 10:31 a.m.
F821 undefined name 'urlopen'	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (112 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (113 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (106 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (120 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:31 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (121 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (112 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (113 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (106 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (103 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (99 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (105 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (109 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 14, 2020, 10:31 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:38 a.m.
E501 line too long (120 > 79 characters)	reviewbot	Oct. 14, 2020, 10:38 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:38 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:39 a.m.
E501 line too long (121 > 79 characters)	reviewbot	Oct. 14, 2020, 10:39 a.m.
E226 missing whitespace around arithmetic operator	reviewbot	Oct. 14, 2020, 10:39 a.m.
E501 line too long (83 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (112 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (113 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (106 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (103 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (99 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (105 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (118 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (109 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (84 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (117 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (110 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (103 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (86 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (87 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (95 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (108 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E501 line too long (114 > 79 characters)	reviewbot	Oct. 14, 2020, 10:34 a.m.
E127 continuation line over-indented for visual indent	reviewbot	Oct. 14, 2020, 10:50 a.m.
E122 continuation line missing indentation or outdented	reviewbot	Oct. 14, 2020, 10:51 a.m.
E122 continuation line missing indentation or outdented	reviewbot	Oct. 14, 2020, 10:51 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (94 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (95 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (88 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (114 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (80 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (86 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (99 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (90 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (89 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
W291 trailing whitespace	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (98 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (91 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.
E501 line too long (84 > 79 characters)	reviewbot	Oct. 14, 2020, 10:47 a.m.
E128 continuation line under-indented for visual indent	reviewbot	Oct. 14, 2020, 10:47 a.m.

flake8 failed.

JSHint passed.

flake8

rbintegrations/matrix/integration.py (Diff revision 1)
Show all issues
```
F401 'json' imported but unused
```

rbintegrations/matrix/integration.py (Diff revision 1)

Show all issues

F401 'django.utils.six.moves.urllib.request.Request' imported but unused

rbintegrations/matrix/integration.py (Diff revision 1)

Show all issues

F401 'django.utils.six.moves.urllib.request.urlopen' imported but unused

rbintegrations/matrix/integration.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 1)
Show all issues
```
E303 too many blank lines (3)
```
rbintegrations/matrix/integration.py (Diff revision 1)
Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 1)
Show all issues
```
E501 line too long (80 > 79 characters)
```

Commits:

	Summary	Author
-	[WIP] - Add Matrix Chat option to Integration	ruonan
+	[WIP] - Add Matrix Chat option to Integration	ruonan
+	[WIP] implementing matrix communication	ruonan
+	[WIP] got matrix integration workinggit add .	ruonan

Diff:

Revision 2 (+3846 -16)

Show changes

	rbintegrations/extension.py
	rbintegrations/matrix/__init__.py
	rbintegrations/matrix/forms.py
	rbintegrations/matrix/integration.py
	rbintegrations/matrix/tests.py
	rbintegrations/static/images/matrix/icon.png

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

rbintegrations/matrix/forms.py (Diff revision 2)
Show all issues
```
E303 too many blank lines (2)
```

rbintegrations/matrix/integration.py (Diff revision 2)

Show all issues

F401 'django.utils.six.moves.urllib.request.Request' imported but unused

rbintegrations/matrix/integration.py (Diff revision 2)

Show all issues

F401 'django.utils.six.moves.urllib.request.urlopen' imported but unused

rbintegrations/matrix/integration.py (Diff revision 2)

Show all issues

F841 local variable 'response' is assigned to but never used

rbintegrations/matrix/integration.py (Diff revision 2)
Show all issues
```
E302 expected 2 blank lines, found 1
```
rbintegrations/matrix/integration.py (Diff revision 2)
Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 2)
Show all issues
```
E501 line too long (80 > 79 characters)
```

Commits:

	Summary	Author
-	[WIP] - Add Matrix Chat option to Integration	ruonan
-	[WIP] implementing matrix communication	ruonan
-	[WIP] got matrix integration workinggit add .	ruonan
+	added Matrix integration option	ruonan

Diff:

Revision 3 (+2596 -4)

Show changes

	rbintegrations/extension.py
	rbintegrations/basechat/integration.py
	rbintegrations/matrix/__init__.py
	rbintegrations/matrix/forms.py
	rbintegrations/matrix/integration.py
	rbintegrations/matrix/tests.py
	rbintegrations/static/images/matrix/icon.png

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 98 failures.
```
rbintegrations/basechat/integration.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/forms.py (Diff revision 3)
Show all issues
```
E501 line too long (86 > 79 characters)
```
rbintegrations/matrix/forms.py (Diff revision 3)
Show all issues
```
E501 line too long (94 > 79 characters)
```
rbintegrations/matrix/forms.py (Diff revision 3)
Show all issues
```
E501 line too long (82 > 79 characters)
```
rbintegrations/matrix/forms.py (Diff revision 3)
Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
F401 'matrix_client.api.MatrixHttpApi' imported but unused
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E501 line too long (108 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E501 line too long (121 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E302 expected 2 blank lines, found 1
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
W293 blank line contains whitespace
```
rbintegrations/matrix/integration.py (Diff revision 3)
Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 3)
Show all issues
```
E501 line too long (103 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 3)
Show all issues
```
E501 line too long (118 > 79 characters)
```

Summary:

-	[WIP] - Add Matrix Chat option to Integration
+	Add Matrix Chat option to Integration

Description:

~		[WIP] - Add Matrix Chat option to Integration
	~	Admins can choose to add Matrix Integration. This will send notifications to the registered matrix room about new review requets and updates to existing ones.

Testing Done:

	+	Tested in RB4.
	+	Edited Slack unit test cases to test Matrix integration. All tests have passed except 2 of then complaining about missing image attachment
	+	tests with errors are:
	+	- test_notify_new_review_request_with_image_file_attachment
	+	- test_notify_updated_review_request_with_new_image_attachments

Commits:

	Summary	Author
-	added Matrix integration option	ruonan
+	add Matrix integration option	ruonan

Diff:

Revision 4 (+2692 -2)

Show changes

	.idea/.gitignore
	.idea/misc.xml
	.idea/modules.xml
	.idea/rbintegrations.iml
	.idea/vcs.xml
	.idea/inspectionProfiles/profiles_settings.xml
	docs/releasenotes/conf.py
	rbintegrations/extension.py
	6 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 90 failures.
```
docs/releasenotes/conf.py (Diff revision 4)
Show all issues
```
F821 undefined name 'self'
```

docs/releasenotes/conf.py (Diff revision 4)

Show all issues

E126 continuation line over-indented for hanging indent

docs/releasenotes/conf.py (Diff revision 4)
Show all issues
```
F821 undefined name 'json'
```
docs/releasenotes/conf.py (Diff revision 4)
Show all issues
```
F821 undefined name 'urlopen'
```
docs/releasenotes/conf.py (Diff revision 4)
Show all issues
```
E501 line too long (83 > 79 characters)
```
docs/releasenotes/conf.py (Diff revision 4)
Show all issues
```
E501 line too long (112 > 79 characters)
```
docs/releasenotes/conf.py (Diff revision 4)
Show all issues
```
E501 line too long (113 > 79 characters)
```
docs/releasenotes/conf.py (Diff revision 4)
Show all issues
```
E501 line too long (106 > 79 characters)
```
docs/releasenotes/conf.py (Diff revision 4)
Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 4)
Show all issues
```
E501 line too long (83 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 4)
Show all issues
```
E501 line too long (120 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 4)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 4)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 4)
Show all issues
```
E501 line too long (121 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 4)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (83 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (112 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (113 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (106 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (118 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (103 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (99 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (105 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (118 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (109 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 4)
Show all issues
```
E501 line too long (108 > 79 characters)
```

Summary:

-	Add Matrix Chat option to Integration
+	Add Matrix option to Integration

Commits:

	Summary	Author
-	add Matrix integration option	ruonan
+	add Matrix integration option	ruonan

Diff:

Revision 5 (+2668 -2)

Show changes

	.idea/.gitignore
	.idea/misc.xml
	.idea/modules.xml
	.idea/rbintegrations.iml
	.idea/vcs.xml
	.idea/inspectionProfiles/profiles_settings.xml
	rbintegrations/extension.py
	rbintegrations/basechat/integration.py
	5 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 81 failures.
```
rbintegrations/matrix/integration.py (Diff revision 5)
Show all issues
```
E501 line too long (83 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 5)
Show all issues
```
E501 line too long (120 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 5)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 5)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/integration.py (Diff revision 5)
Show all issues
```
E501 line too long (121 > 79 characters)
```
rbintegrations/matrix/integration.py (Diff revision 5)
Show all issues
```
E226 missing whitespace around arithmetic operator
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (83 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (112 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (113 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (106 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (118 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (103 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (99 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (105 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (118 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (109 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (108 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (84 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (117 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (110 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (103 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (86 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (87 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (95 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (108 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 5)
Show all issues
```
E501 line too long (114 > 79 characters)
```

Commits:

	Summary	Author
-	add Matrix integration option	ruonan
+	add Matrix integration option	ruonan

Diff:

Revision 6 (+2712 -2)

Show changes

	.idea/.gitignore
	.idea/misc.xml
	.idea/modules.xml
	.idea/rbintegrations.iml
	.idea/vcs.xml
	.idea/inspectionProfiles/profiles_settings.xml
	rbintegrations/extension.py
	rbintegrations/basechat/integration.py
	5 more

Checks run (1 failed, 1 succeeded)

flake8 failed.

JSHint passed.

flake8

```
Warning: Showing 30 of 144 failures.
```
rbintegrations/matrix/integration.py (Diff revision 6)
Show all issues
```
E127 continuation line over-indented for visual indent
```
rbintegrations/matrix/integration.py (Diff revision 6)
Show all issues
```
E122 continuation line missing indentation or outdented
```
rbintegrations/matrix/integration.py (Diff revision 6)
Show all issues
```
E122 continuation line missing indentation or outdented
```
rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (94 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (95 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (88 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
W291 trailing whitespace
```
rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (114 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (80 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
W291 trailing whitespace
```

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (80 > 79 characters)
```

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (86 > 79 characters)
```

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (99 > 79 characters)
```

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (90 > 79 characters)
```

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (89 > 79 characters)
```
rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
W291 trailing whitespace
```

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (98 > 79 characters)
```

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (91 > 79 characters)
```

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

rbintegrations/matrix/tests.py (Diff revision 6)
Show all issues
```
E501 line too long (84 > 79 characters)
```

rbintegrations/matrix/tests.py (Diff revision 6)

Show all issues

E128 continuation line under-indented for visual indent

Review Board 6.0 beta 3 (dev)

Add Matrix option to Integration

flake8

Commits:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commits:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Summary:

Description:

Testing Done:

Commits:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Summary:

Commits:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Commits:

Diff:

Checks run (1 failed, 1 succeeded)

flake8

Status: Discarded