Skip to content

change: 提前判断议题是否与发布有关#127

Merged
he0119 merged 2 commits intomainfrom
change/publish-type
Apr 10, 2023
Merged

change: 提前判断议题是否与发布有关#127
he0119 merged 2 commits intomainfrom
change/publish-type

Conversation

@he0119
Copy link
Copy Markdown
Member

@he0119 he0119 commented Apr 10, 2023

因为就算延后执行,议题的名称也不会变。这样如果议题与发布无关就可直接跳过,少执行一次请求。

因为就算延后执行,议题的名称也不会变
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 10, 2023

Codecov Report

Merging #127 (63fc111) into main (7faced2) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #127      +/-   ##
==========================================
+ Coverage   95.62%   95.65%   +0.02%     
==========================================
  Files           7        7              
  Lines         572      575       +3     
==========================================
+ Hits          547      550       +3     
  Misses         25       25              
Impacted Files Coverage Δ
src/plugins/publish/__init__.py 97.89% <100.00%> (-0.03%) ⬇️
src/plugins/publish/depends.py 96.87% <100.00%> (+0.44%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@he0119 he0119 merged commit 32af040 into main Apr 10, 2023
@he0119 he0119 deleted the change/publish-type branch April 10, 2023 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant