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

fixup '*' -> '/.*/' for CI filter #4059

Merged
merged 1 commit into from
Jun 15, 2021
Merged

fixup '*' -> '/.*/' for CI filter #4059

merged 1 commit into from
Jun 15, 2021

Conversation

mattip
Copy link
Contributor

@mattip mattip commented Jun 15, 2021

Looking at pytorch/pytorch, the syntax for the catch-all filter seems off. And indeed, it seems the doc build is not triggering

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Looks like it got properly triggered on this PR now. Thanks @mattip !

@fmassa fmassa merged commit 496cb40 into pytorch:master Jun 15, 2021
@github-actions
Copy link

Hey @NicolasHug, @fmassa!

You approved or merged this PR, but no labels were added.

facebook-github-bot pushed a commit that referenced this pull request Jun 21, 2021
Reviewed By: fmassa

Differential Revision: D29264316

fbshipit-source-id: 9220ed7d303b1994e9056509c722f261c3d6356f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants