Skip to content

Fix linting workflow - #107

Merged
mattwthompson merged 4 commits into
mainfrom
fix-lint
Sep 3, 2025
Merged

mattwthompson merged 4 commits into
mainfrom
fix-lint

Conversation

@mattwthompson

Copy link
Copy Markdown
Member

This has been broken for some time and I get nightly notifications about it

https://github.com/openforcefield/openff-docs/actions/workflows/lint.yaml

@github-actions

github-actions Bot commented Sep 2, 2025

Copy link
Copy Markdown

This repository caches data generated from notebooks stored in other repositories in the _cookbook_data_* branches. Regenerating this cache takes about half an hour, so it is not done automatically in PRs. The current value of the cache from the main branch has been copied over to this PR's cache.

To regenerate the cache for this PR, create a comment on this PR consisting only of the string:

/regenerate-cache

@mattwthompson

Copy link
Copy Markdown
Member Author

/regenerate-cache

@github-actions

github-actions Bot commented Sep 2, 2025

Copy link
Copy Markdown

A workflow has been dispatched to regenerate the cookbook cache for this PR.

  • Run ID: 17410028906
  • Triggering actor: github-actions[bot]
  • Target branch: fix-lint
  • Deployment branch: _cookbook_data_PR107

@github-actions

github-actions Bot commented Sep 2, 2025

Copy link
Copy Markdown

A workflow dispatched to regenerate the cookbook cache for this PR has just finished.

  • Run ID: 17410028906

  • Triggering actor: github-actions[bot]

  • Target branch: fix-lint

  • Deployment branch: _cookbook_data_PR107

  • Job status: success

  • Notebooks status: 29 / 29 notebooks successfully executed (0 failures ignored)

    • [ignored]

Changes will only be visible in the ReadTheDocs preview after it has been rebuilt. You will need to be logged in to ReadTheDocs to see the rebuild option.

@mattwthompson
mattwthompson marked this pull request as ready for review September 2, 2025 17:05

@Yoshanuikabundi Yoshanuikabundi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One blocking typo. I am still not getting notifications for these failures. https://docs.github.com/en/actions/concepts/workflows-and-actions/notifications-for-workflow-runs says that notifications go to the user who created the workflow, or to the user who last changed the chron syntax, or to the user who last enabled the action, so I'll try disabling and reenabling all the workflows in this repo.

Comment thread source/install.md Outdated
Co-authored-by: Josh A. Mitchell <yoshanuikabundi@gmail.com>
@mattwthompson
mattwthompson merged commit 97a9049 into main Sep 3, 2025
2 checks passed
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.

2 participants