Ludwig Nussel
4e03967dba
Merge pull request #1455 from lnussel/leaper
...
leaper: allow comment override also on openSUSE
2018-03-23 11:20:36 +01:00
Ludwig Nussel
60bb341c29
leaper: allow comment override also on openSUSE
2018-03-19 16:29:33 +01:00
Jimmy Berry
39ef75493d
Change references to osc-plugin-factory to openSUSE-release-tools.
2018-03-16 14:58:27 -05:00
Jimmy Berry
16f49c7518
leaper: do not decline SLE requests, but rather ask managers for input.
2018-03-07 15:11:02 -06:00
Jimmy Berry
8332692dad
leaper: move review groups to osclib.conf.
2018-03-07 14:55:09 -06:00
Ludwig Nussel
58bd9c70e1
leaper: allow turning FORK into something else
2018-02-13 12:57:26 +01:00
Ludwig Nussel
4e3148aace
leaper: don't check maintainer for other requests
2018-02-12 13:27:27 +01:00
Ludwig Nussel
d4a4cf8396
leaper: switch off maintainer review for updates from Factory
2018-02-09 13:31:11 +01:00
Jimmy Berry
f98a78c1ae
Remove Vim config in favor of EditorConfig.
2018-01-23 00:48:00 -06:00
Jimmy Berry
8c4891bd95
ReviewBot: replace get_devel_project() with osclib.core.devel_project_get().
2018-01-17 18:09:30 -06:00
Ludwig Nussel
981fe38696
Skip maintainer review for certain Leap 15 submissions
2017-11-20 14:54:49 +01:00
d1c4801578
Another set of low-noise flake8 cleanups
2017-10-20 20:46:44 +02:00
3cf49d9556
Python 2.6+ / 3.x style except clauses
...
This newer style except statement allows for compatibility with
Python 3.x and does not harm on python 2.6+.
2017-10-16 22:47:58 +02:00
Ludwig Nussel
30930ba7fd
leaper: ignore 404
...
Some projects may not exist yet, such as SLE :Update.
Don't bail out because of that.
2017-10-05 16:44:22 +02:00
Ludwig Nussel
dd8adeb97d
leaper: fix iterating through list
2017-09-07 16:27:47 +02:00
Ludwig Nussel
021c6f5da0
leaper: avoid comments about unhandled request types
...
delete and set_bugowner are quite common, so commenting about them
doesn't add much value.
2017-08-29 11:14:18 +02:00
Ludwig Nussel
635f41b18c
leaper: fix syntax error
2017-08-29 11:14:10 +02:00
Jimmy Berry
078ec68697
ReviewBot: add_review(): provide allow_duplicate parameter and default of False.
...
Also update calls based on default behavior change. As described in the
comment above add_review() there seems to be no reason to ever add a
duplicate review.
The check in leaper would skip unless state == declined. In such a case
the review would automatically be reopened along with the request so that
logic is unnecessary.
2017-08-15 22:18:53 -05:00
Jimmy Berry
ef93709e16
ReviewBot: add_review(): raise exception instead of returning boolean.
...
This ensures that calling bots crash rather than continue which can cause
non-atomic state changes that miss reviews when OBS crashes or is
unavailable.
2017-08-15 21:43:02 -05:00
Jimmy Berry
0aaa2ea02d
leaper: transition from SLE-12-SP3 and Leap:42.3 to SLE-15 and Leap:15.0.
2017-08-01 16:42:16 -05:00
Jimmy Berry
060b50f072
ReviewBot: add comment_handler option and use in leaper.
2017-07-12 15:31:30 -05:00
Jimmy Berry
2c1a21704d
ReviewBot: copy default review messages before each review.
2017-07-12 15:31:30 -05:00
Jimmy Berry
3b1c11f45c
ReviewBot: provide only_one_action option and use in leaper and check_source.
2017-07-12 15:31:30 -05:00
Jimmy Berry
7bcf94a35f
ReviewBot: provide request_default_return option and use in leaper.
2017-07-12 15:31:30 -05:00
Ludwig Nussel
1c7a37bc0e
Turn off automatic submission from Factory
2017-07-03 17:23:00 +02:00
Jimmy Berry
c1dc56c251
leaper: correct invalid diff link when package present in SLE.
2017-06-27 01:15:25 -05:00
Ludwig Nussel
e2cd308062
Merge pull request #890 from lnussel/leaper
...
Add more klickable links
2017-06-19 16:49:27 +02:00
Jimmy Berry
d804a9d333
leaper: drop extraneous _ prefix to src_srcinfo during in_sle_origin check.
2017-05-11 23:34:35 -05:00
Jimmy Berry
f203f0c5e0
leaper: indicate if submission is already accepted to SLE origin.
...
It seems unlikely that checking for requests would be acceptable for SLE.
2017-05-09 20:18:23 -05:00
Ludwig Nussel
18fd77c51f
Add more klickable links
2017-05-09 17:33:45 +02:00
Jimmy Berry
585a9d1f84
leaper: do not add review team for SLE requests for FORK'd packages.
2017-04-20 22:18:25 -05:00
Ludwig Nussel
3029046697
Add clickable link to groups
2017-04-18 16:57:21 +02:00
Ludwig Nussel
cfc57126e1
Refactor adding group reviews
2017-04-18 16:51:54 +02:00
Ludwig Nussel
6de8f95655
Add legal review for forks
2017-04-18 16:38:48 +02:00
Jimmy Berry
5f77f48437
leaper: reset automatic_submission to False on each review.
...
Perhaps this should be handled on a larger scale where initial variable
states are not set in __init__ but rather in special method or reset in
some manor.
2017-04-10 23:17:09 -05:00
Ludwig Nussel
67daf27c59
Merge pull request #760 from lnussel/reviewoptions
...
Add options for review team an release managers
2017-03-16 10:40:28 +01:00
Jimmy Berry
21b887f5f2
leaper: allow self submission.
2017-03-15 15:08:21 -05:00
Ludwig Nussel
1af91580ac
Add options for review team an release managers
2017-03-15 17:32:48 +01:00
Ludwig Nussel
bc7ee557fb
Also mention where a package came from in 42.1
2017-03-14 13:51:46 +01:00
Ludwig Nussel
cad7c8520f
Correctly report unchanged devel project
...
Refactor check for same origin into separate function
2017-03-14 13:51:39 +01:00
Ludwig Nussel
96136f324e
Add note if a package is also in SLE
2017-03-14 13:51:33 +01:00
Jimmy Berry
134d340dcf
leaper: notify submit request authors of automatic submission.
2017-02-17 09:56:41 -06:00
Jimmy Berry
748fce38cc
ReviewBot & leaper: provide deduplicate method for leaper comment.
...
Otherwise, leaper will indicate an unhandled request twice.
2017-02-15 10:01:20 -06:00
Jimmy Berry
dcf435430c
leaper: utilize ReviewBot check_action__default() message.
2017-02-15 09:45:18 -06:00
Jimmy Berry
b0f4a5fe2b
leaper: drop extra definition of check_action__default().
...
The second definition being the current behavior and added later would seem
to be the desired implementation.
2017-02-15 09:05:45 -06:00
Ludwig Nussel
5b61b92808
Merge pull request #684 from jberry-suse/leaper-comment-to-reviewbot
...
ReviewBot: refactor leaper comment from log functionality.
2017-02-15 09:59:32 +01:00
Jimmy Berry
92a5f60369
ReviewBot: use super().check_source_submission() in subclasses.
...
No need to duplicate the log message in multiple places.
2017-02-10 16:09:51 -06:00
Jimmy Berry
015d59cdc7
ReviewBot: refactor leaper comment from log functionality.
...
Simple interface:
- comment_handler_add(): Add handler to start recording log messages
for comment.
- comment_write(): Write comment from log messages if not similar to
previous comment.
See leaper.py for example usage.
2017-02-10 15:58:57 -06:00
Ludwig Nussel
cb02316478
Put links to source diffs
...
Easily clickable links are a convenient feature.
2017-01-31 17:11:33 +01:00
Ludwig Nussel
f4406fb2b2
explicitly mention if a package exists with different sources
2017-01-26 13:18:38 +01:00