Skip to content

Commit f5ea74a

Browse files
authored
Merge pull request #3759 from migueldiascosta/missing_patches
warn about potentially missing patches in --new-pr
2 parents 23071bd + 72dbc89 commit f5ea74a

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

easybuild/tools/github.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1783,6 +1783,14 @@ def new_pr(paths, ecs, title=None, descr=None, commit_msg=None):
17831783
res = new_branch_github(paths, ecs, commit_msg=commit_msg)
17841784
file_info, deleted_paths, _, branch_name, diff_stat, pr_target_repo = res
17851785

1786+
for ec, ec_path in zip(file_info['ecs'], file_info['paths_in_repo']):
1787+
for patch in ec.asdict()['patches']:
1788+
if isinstance(patch, tuple):
1789+
patch = patch[0]
1790+
if patch not in paths['patch_files'] and not os.path.isfile(os.path.join(os.path.dirname(ec_path), patch)):
1791+
print_warning("new patch file %s, referenced by %s, is not included in this PR" %
1792+
(patch, ec.filename()))
1793+
17861794
new_pr_from_branch(branch_name, title=title, descr=descr, pr_target_repo=pr_target_repo,
17871795
pr_metadata=(file_info, deleted_paths, diff_stat), commit_msg=commit_msg)
17881796

0 commit comments

Comments
 (0)