Skip to content

Conversation

@mattip
Copy link
Contributor

@mattip mattip commented Jun 14, 2021

Continuation of #3998. In that PR, build_docs was mistakenly restricted to nightly and tags, see this comment

@fmassa

@fmassa
Copy link
Member

fmassa commented Jun 14, 2021

Weird, I still don't see the build_docs job running in this PR.

@NicolasHug do you know what might be happening?

@NicolasHug
Copy link
Member

I don't know for sure, but maybe this is one of these cases where we need to merge the PR to see the effective changes?

branches:
only:
- nightly
- '*'
Copy link
Member

Choose a reason for hiding this comment

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

Do you think the doc job is not being run because we have ' around the *?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The filter with the '*' is the same as the base job build_docs runs off (binary_linux_wheel_py3.7_cpu) which works fine.

      - binary_linux_wheel:
          conda_docker_image: pytorch/conda-builder:cpu
          cu_version: cpu
          filters:
            branches:
              only: '*'
            tags:
              only: /v[0-9]+(\.[0-9]+)*-rc[0-9]+/
          name: binary_linux_wheel_py3.7_cpu
          python_version: '3.7'
          wheel_docker_image: pytorch/manylinux-cuda102

@malfet malfet merged commit 5d93a87 into pytorch:master Jun 14, 2021
@github-actions
Copy link

Hey @malfet!

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

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

Differential Revision: D29105973

fbshipit-source-id: 4179ab333f82d99765ee594eebcf81b4d4c1715e
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.

5 participants