Skip to content

Commit 0fbd42a

Browse files
authored
Release helper optimize (Azure#27355)
* optimize * optimize * optimize
1 parent 74fe7f2 commit 0fbd42a

File tree

2 files changed

+21
-16
lines changed

2 files changed

+21
-16
lines changed

scripts/release_helper/python.py

Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -102,24 +102,29 @@ def auto_reply(self) -> None:
102102
if self.issue_package.issue.comments == 0 or _CONFIGURED in self.issue_package.labels_name:
103103
issue_number = self.issue_package.issue.number
104104
if not self.readme_comparison:
105-
issue_link = self.issue_package.issue.html_url
106-
release_pipeline_url = get_python_release_pipeline(self.output_folder)
107-
res_run = run_pipeline(issue_link=issue_link,
108-
pipeline_url=release_pipeline_url,
109-
spec_readme=self.readme_link + '/readme.md',
110-
python_tag=self.python_tag,
111-
rest_repo_hash=self.rest_repo_hash
112-
)
113-
if res_run:
114-
self.log(f'{issue_number} run pipeline successfully')
115-
if _CONFIGURED in self.issue_package.labels_name:
116-
self.issue_package.issue.remove_from_labels(_CONFIGURED)
117-
else:
118-
self.log(f'{issue_number} run pipeline fail')
119-
self.add_label(_AUTO_ASK_FOR_CHECK)
105+
try:
106+
issue_link = self.issue_package.issue.html_url
107+
release_pipeline_url = get_python_release_pipeline(self.output_folder)
108+
res_run = run_pipeline(issue_link=issue_link,
109+
pipeline_url=release_pipeline_url,
110+
spec_readme=self.readme_link + '/readme.md',
111+
python_tag=self.python_tag,
112+
rest_repo_hash=self.rest_repo_hash
113+
)
114+
if res_run:
115+
self.log(f'{issue_number} run pipeline successfully')
116+
else:
117+
self.log(f'{issue_number} run pipeline fail')
118+
except Exception as e:
119+
self.comment(f'hi @{self.assignee}, please check release-helper: `{e}`')
120+
if _AUTO_ASK_FOR_CHECK not in self.issue_package.labels_name:
121+
self.add_label(_AUTO_ASK_FOR_CHECK)
120122
else:
121123
self.log(f'issue {issue_number} need config readme')
122124

125+
if _CONFIGURED in self.issue_package.labels_name:
126+
self.issue_package.issue.remove_from_labels(_CONFIGURED)
127+
123128
def attention_policy(self):
124129
if _BRANCH_ATTENTION in self.issue_package.labels_name:
125130
self.bot_advice.append('new version is 0.0.0, please check base branch!')

scripts/release_helper/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
def get_origin_link_and_tag(issue_body_list: List[str]) -> (str, str):
2727
link, readme_tag = '', ''
2828
for row in issue_body_list:
29-
if 'link' in row.lower() and link == '':
29+
if 'link' in row.lower() and 'release request' not in row.lower() and link == '':
3030
link = row.split(":", 1)[-1].strip()
3131
if 'readme tag' in row.lower() and readme_tag == '':
3232
readme_tag = row.split(":", 1)[-1].strip()

0 commit comments

Comments
 (0)