Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: fix contains check #28666

Merged
merged 2 commits into from
Jan 4, 2021
Merged

ci: fix contains check #28666

merged 2 commits into from
Jan 4, 2021

Conversation

xrkffgg
Copy link
Member

@xrkffgg xrkffgg commented Jan 4, 2021

github 的自带函数不区分大小写校验

导致 #28659 校验 IE 通过

@kerm1it 有没有好的校验单个 IE 的办法,用 split(' ') ?但有人会用 IE11 之类的,所以还是决定直接校验大写 'IE'

@ant-design-bot
Copy link
Contributor

ant-design-bot commented Jan 4, 2021

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 4, 2021

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 5520cb8:

Sandbox Source
antd reproduction template Configuration

@xrkffgg xrkffgg requested a review from kerm1it January 4, 2021 08:39
@github-actions
Copy link
Contributor

github-actions bot commented Jan 4, 2021

Size Change: 0 B

Total Size: 797 kB

ℹ️ View Unchanged
Filename Size Change
./dist/antd-with-locales.min.js 317 kB 0 B
./dist/antd.compact.min.css 66.9 kB 0 B
./dist/antd.dark.min.css 68.1 kB 0 B
./dist/antd.min.css 66.9 kB 0 B
./dist/antd.min.js 278 kB 0 B

compressed-size-action

@afc163
Copy link
Member

afc163 commented Jan 4, 2021

IE9 || IE 9 || IE10 || IE 10 || IE11 || IE 11

这样是不是够用了。

@kerm1it
Copy link
Member

kerm1it commented Jan 4, 2021

可以使用正则表达式么?例如 title.match

@xrkffgg
Copy link
Member Author

xrkffgg commented Jan 4, 2021

IE9 || IE 9 || IE10 || IE 10 || IE11 || IE 11

够了,11 以后 再无 IE 😂

@xrkffgg
Copy link
Member Author

xrkffgg commented Jan 4, 2021

可以使用正则表达式么?例如 title.match

这个主要是写在 yml 配置里,怕不支持太复杂的语法。如果放到 js 里,就有很多办法了

@xrkffgg xrkffgg merged commit b5aeced into master Jan 4, 2021
@xrkffgg xrkffgg deleted the xrkffgg-patch-1 branch January 4, 2021 08:58
@@ -60,7 +60,7 @@ jobs:
* Ant Design Pro Preview : https://prosite.z23.web.core.windows.net

- name: check-ie
if: contains(github.event.issue.body, 'ant-design-issue-helper') == true && (contains(github.event.issue.title, 'IE') == true || contains(github.event.issue.title, 'Internet Explorer') == true)
if: contains(github.event.issue.body, 'ant-design-issue-helper') == true && github.event.issue.title.includes('IE9') == true || github.event.issue.title.includes('IE 9') == true || github.event.issue.title.includes('IE10') == true || github.event.issue.title.includes('IE 10') == true || github.event.issue.title.includes('IE11') == true || github.event.issue.title.includes('IE 11') == true || contains(github.event.issue.title, 'Internet Explorer') == true)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这样精确一点,body 里有也可以。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

刚看到 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants