yudaike/openSUSE-release-tools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
6,864 Commits 2 Branches 12 Tags
Commit Graph

166 Commits

Author SHA1 Message Date
Stephan Kulow
a80184d44e Add ReviewBot to check for bugowner lines in requests for new packages 2022-03-10 10:41:04 +01:00
Stephan Kulow
d6fb5d680c Remove sentry integration 
It was introduced by #2159 because the various machines were hard to
monitor. Since we moved to botmaster there is no longer a problem with
that and if it ever served us is hard to say as the sentry project's
password is lost for years. So simplify the exception handling by just
throwing it
 2022-02-21 14:41:33 +01:00
Stephan Kulow
aaab4af930 Target F401 - Unused imports 2022-02-18 18:29:27 +01:00
Stephan Kulow
f86579642b Target E501 - Line too long 
https://www.flake8rules.com/rules/E501.html - but 79 characters is a
joke
 2022-02-18 18:17:27 +01:00
Stephan Kulow
f38bd3a7ed Target E128 - Continuation line under-indented for visual indent 
https://www.flake8rules.com/rules/E128.html
 2022-02-18 17:35:33 +01:00
Stephan Kulow
27c364b0c9 Target E251 - Unexpected spaces around keyword 
https://www.flake8rules.com/rules/E251.html
 2022-02-18 17:28:13 +01:00
Stephan Kulow
d8f5d0874e Remove E201 - Whitespace after '(' 
https://www.flake8rules.com/rules/E201.html
 2022-02-18 17:23:19 +01:00
Stephan Kulow
54079e949f Target E202 - Whitespace before ')' 
https://www.flake8rules.com/rules/E202.html
 2022-02-18 17:18:29 +01:00
Stephan Kulow
529d1dafae Target E302 - add two comment lines before class 2022-02-18 17:15:48 +01:00
Stephan Kulow
579145b52c Target F841 - Remove unused variables 2022-02-18 17:01:38 +01:00
Stephan Kulow
a8623bb8d8 Handle E712 - don't compare with == to Booleans 2022-02-18 16:34:58 +01:00
Stephan Kulow
1383ae9725 Targetting E126 - Continuation line over-indented for hanging indent 
https://www.flake8rules.com/rules/E126.html
 2022-02-18 16:02:08 +01:00
Stephan Kulow
d6628beadf Target E125 - Continuation line with same indent 
https://www.flake8rules.com/rules/E125.html
 2022-02-18 15:52:53 +01:00
Stephan Kulow
2023ccbe6e Target E124 - Closing bracket does not match visual indentation 
https://www.flake8rules.com/rules/E124.html
 2022-02-18 15:52:47 +01:00
Stephan Kulow
66a0f152ff Target E129 - Visually indented line with same indent 
https://www.flake8rules.com/rules/E129.html
 2022-02-18 15:52:47 +01:00
Stephan Kulow
5df665ccc5 Targeting E123 - Closing bracket does not match indentation 
https://www.flake8rules.com/rules/E123.html
 2022-02-18 15:52:47 +01:00
Stephan Kulow
e52dc6ce2b Harmonize imports on using lxml for XML parsing 2022-02-18 11:16:01 +01:00
Stephan Kulow
a8f7146baf Remove E713 offense from flaker 2021-09-21 14:20:08 +02:00
Ludwig Nussel
60e5fd94a2 abi-checker: recognize staging 2019-12-19 14:13:48 +01:00
Dirk Mueller
ad06348dcf More flake8 cleanups 2019-12-10 12:50:57 +01:00
Dirk Mueller
ea3b07568c Fix E203 
This fixes incorrect whitespacing inside dictionaries.
 2019-12-10 08:54:32 +01:00
Dirk Mueller
ec88760187 Fix Flake E241 
One more step towards white space sanity. Next group is E20x.
 2019-12-10 08:46:02 +01:00
Dirk Mueller
32ad0adc2d Fix all E22x errors 
This cleans up white spaces around various kinds of operators to be
consistent.
 2019-11-27 11:08:06 +01:00
Jimmy Berry
a3761c62e5 ReviewBot: include request description in search for comment commands. 2019-11-04 13:22:38 -06:00
Jimmy Berry
701021070c ReviewBot: rework override check to operate on actions. 
In multi-action workflows the override check should be performed by
action so that different groups of users can override for different
actions. Additionally, abstracting the request_commands() method provides
a flexible base for additional commands to be added by ReviewBots.
 2019-11-04 13:22:38 -06:00
Jimmy Berry
7033044469 ReviewBot: provide request_age_wait(). 
Provides configurable default, config key, and config override value.
 2019-09-25 14:17:21 -05:00
Jimmy Berry
0930b8368e ReviewBot: add_review(): handle 403 as expected when superseded/rovoked. 2019-09-18 20:03:41 -05:00
Jimmy Berry
43067cf8e4 ReviewBot: _set_review(): handle review being revoked or superseded. 
This can occur when the review state is changed during the review.
 2019-09-18 18:22:56 -05:00
Jimmy Berry
14971e2db0
Merge pull request #2174 from jberry-suse/sentry-improvements 
sentry improvements and check_source logger error demotion
 2019-08-28 15:34:02 -05:00
Jimmy Berry
208336ca80 ReviewBot: action_method(): drop request.type sentry extra in favor or action.key. 2019-08-27 14:45:29 -05:00
Jimmy Berry
4ca9995b5d ReviewBot: simplify sentry_sdk initialization line since defined globally. 2019-08-27 14:44:54 -05:00
Jimmy Berry
b0d7e9930a ReviewBot: provide request.id and action.key as sentry extras. 2019-08-27 14:44:42 -05:00
Jimmy Berry
ca4033e614 ReviewBot: action_method(): reduce sentry extra to filter(None, ...). 2019-08-27 14:31:00 -05:00
Jimmy Berry
12d15ae850 ReviewBot: reduce unhandled request type log message from error to info. 
Uninteresting request action types are intentionally left to the default
handler and does not constitute and error.
 2019-08-27 08:20:13 -05:00
Jimmy Berry
5dd0a99068 ReviewBot: provide sentry_sdk integration. 2019-08-16 16:01:55 -05:00
Jimmy Berry
cd6b2b3c33 ReviewBot: nest review messages by action key when multiple actions. 
Not only does this expose previously hidden messages on multi-action
requests, but also provides clarity as to which action triggered a
specific response. Since the keys are generated in a standard way and
the data formatted as YAML it can also be retrieved.
 2019-07-16 10:27:37 -05:00
Jimmy Berry
a8ff27b81f osclib/core: provide action_is_patchinfo() from ReviewBot. 
Allows for usage outside of ReviewBot children.
 2019-07-11 16:01:42 -05:00
Stephan Kulow
011c80cc2a Remove debug output left in ReviewBot 2019-05-27 18:49:20 +02:00
Stephan Kulow
fe38a6127d Exit with an error if there were exceptions caught 
Fixes #2041
 2019-05-27 13:35:59 +02:00
Stephan Kulow
606e498554 ReviewBot: Simplify the !interval case 
We only want to run the workfunc() and that's it
 2019-05-27 11:03:04 +02:00
Stephan Kulow
3f3bbb3e10 Run even more bots in python3 2019-05-16 06:59:25 +02:00
Stephan Kulow
595e84191e More porting to python3 
Travis changed default python to 3.6, which has impact on the way
flake8 checks. So make sure we run fine under flake8 of python 3
 2019-04-20 21:16:50 +02:00
Jimmy Berry
d0e0f2aa3a ReviewBot: handle any exception raised by check_one_request(). 
Prevents a single failure from stopping all reviews.
 2019-03-29 14:31:44 -05:00
Jimmy Berry
e930ea37c3 ReviewBot: request_override_check(): default to accept. 2019-02-26 09:33:35 -06:00
Jimmy Berry
d32b0c65bb ReviewBot: request_override_check(): provide force option. 2019-02-26 09:33:14 -06:00
Ludwig Nussel
ee7cdc2a90 ReviewBot: catch exception for adding devel project review 
if the bot is run on requests it has no role it, adding devel project
review may fail. So just handle that and proceed.
 2019-02-05 08:25:14 +01:00
Ludwig Nussel
9091593263 ReviewBot: refactor logic 2019-01-16 15:19:01 +01:00
Stephan Kulow
ad48ebfe20
Merge pull request #1795 from coolo/dont_error_on_RR 
If we can't resolve the package name, it's no reason to decline
 2018-11-27 06:34:31 +01:00
Stephan Kulow
4d6133a25b If we can't resolve the package name, it's no reason to decline 
Sometimes maintenance adds specific containers to build in the
incident
 2018-11-19 13:00:19 +01:00
Stephan Kulow
2fe9701238 Use more print from the future 2018-11-17 10:16:13 +01:00