Skip to content

fix formatter to not raise on empty header, update build image #855

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

Merged
merged 1 commit into from
Oct 5, 2021
Merged

fix formatter to not raise on empty header, update build image #855

merged 1 commit into from
Oct 5, 2021

Conversation

mattip
Copy link
Contributor

@mattip mattip commented Oct 5, 2021

Closes #853

  • Some files, like README.md and CONTRIBUTOR.md in the _hub subrepo do not have header indexes. Do not bother rendering them through notebook.
  • Update the CircleCI build image

@malfet, @brianjo

@netlify
Copy link

netlify bot commented Oct 5, 2021

👷 Deploy Preview for pytorch-dot-org-preview processing.

🔨 Explore the source changes: 7fc2a85

🔍 Inspect the deploy log: https://app.netlify.com/sites/pytorch-dot-org-preview/deploys/615cb1dc06f1c90008dabca3

@mattip
Copy link
Contributor Author

mattip commented Oct 5, 2021

I don;t see a circleci job on the PR, so I hope it solves the problem. Locally the build script worked. 🤷

@malfet
Copy link
Contributor

malfet commented Oct 5, 2021

I don;t see a circleci job on the PR, so I hope it solves the problem. Locally the build script worked. 🤷

This is sort of expected behaviour per

workflow-build:
jobs:
- build:
filters:
branches:
only: site

@malfet malfet merged commit d2a7f03 into pytorch:site Oct 5, 2021
malfet added a commit that referenced this pull request Oct 6, 2021
Introduced by #855, but its hard to test in PR considering how config.yml is written
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.

builds are failing
3 participants