Tianocore/edk2.git/BaseTools/Scripts/GetMaintainer.py, branch dependabot/github_actions/actions/github-script-9

BaseTools/GetMaintainer.py: Add GitHub username argument

2024-07-25T02:28:49+00:00

Adds a new `-g` parameter so that output will also include the GitHub username. This change uses a simple regular expression as opposed to directly returning the original line from the file to make the extraction of GitHub usernames more robust to other changes on the line in the maintainers text file. Signed-off-by: Michael Kubacki

BaseTools/Scripts/GetMaintainer: Sort output addresses

2023-11-11T02:31:13+00:00

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4593 Sort the list of output addresses alphabetically so this script produces the same output even if the order of patches in a patch series is modified such that that order of files processed by this script changes. Use set() logic instead of OrderedDict to accumulate the list of unique addresses that are sorted alphabetically. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Cc: Yuwei Chen Cc: Leif Lindholm Signed-off-by: Michael D Kinney Acked-by: Rebecca Cran Reviewed-by: Leif Lindholm

BaseTools/Scripts/GetMaintainer: Handle reviewer only case

2023-11-11T02:31:13+00:00

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4593 If a package only has reviewers and no maintainers, then also return the maintainers. In order to detect this case, get_maintainers() is updated to return maintainers, reviews, and lists separately instead of a single merged list. This also allows this module to be used by other scripts that need to distinguish between maintainers, reviewers, and lists. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Cc: Yuwei Chen Cc: Leif Lindholm Signed-off-by: Michael D Kinney Acked-by: Rebecca Cran Reviewed-by: Leif Lindholm

BaseTools/Scripts/GetMaintainer: refactor internal returns as dicts

2023-11-11T02:31:13+00:00

To clean up interfaces, change the lookup functions to return dictionaries rather than multiple values. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Cc: Yuwei Chen Cc: Michael D Kinney Signed-off-by: Leif Lindholm Acked-by: Rebecca Cran Reviewed-by: Michael D Kinney

BaseTools/Scripts/GetMaintainer: Simplify logic

2023-11-11T02:31:13+00:00

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4593 get_section_maintainers() either returns a list with valid entries or an empty list. It never returns None. Simplify logic that accumulates maintainers and lists by unconditionally appending lists returned from get_section_maintainers(). Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Cc: Yuwei Chen Cc: Leif Lindholm Signed-off-by: Michael D Kinney Acked-by: Rebecca Cran Reviewed-by: Leif Lindholm

BaseTools/Scripts/GetMaintainer: Fix logic bug collecting maintainers

2023-11-11T02:31:13+00:00

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4593 Fix logic bug where maintainers is incorrectly added to lists. Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Cc: Yuwei Chen Cc: Leif Lindholm Signed-off-by: Michael D Kinney Acked-by: Rebecca Cran Reviewed-by: Leif Lindholm

BaseTools/Scripts: Allow GitHub ID at end Maintainers.txt lines

2021-07-22T07:06:48+00:00

* Update GetMaintainer.py to support an optional GitHub ID at the end of maintainer and reviewer lines. * Remove contents after email address from standard output * Fix minor issue in --lookup to convert file path separators from '\' to '/' to be compatible with regular expression file matching. Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen Signed-off-by: Michael D Kinney Reviewed-by: Bob Feng

BaseTools/Scripts: Fix GetMaintainer.py line endings

2021-07-22T07:06:48+00:00

Cc: Bob Feng Cc: Liming Gao Cc: Yuwei Chen Signed-off-by: Michael D Kinney Reviewed-by: Bob Feng

BaseTools: add handling for 'S:' flag to GetMaintainer.py

2020-05-08T04:37:08+00:00

GetMaintainer.py already extracts the value of any S: tags for sections, but it doesn't do anything with that information. Print a warning message, with the status, for each matching section with a status explicitly set to anything other than 'Supported' or 'Maintained'. Cc: Bob Feng Cc: Liming Gao Signed-off-by: Leif Lindholm Acked-by: Ard Biesheuvel Reviewed-by: Bob Feng

BaseTools: add GetMaintainer.py script

2019-07-24T16:45:55+00:00

Add a new script GetMaintainer.py that uses the new Maintainer.txt format to determine which addresses to cc on patch submission. Signed-off-by: Leif Lindholm Reviewed-by: Philippe Mathieu-Daude Tested-by: Philippe Mathieu-Daude Acked-by: Laszlo Ersek Acked-by: Liming Gao Reviewed-by: Bob Feng