Skip to content

Conversation

@jonathangreen
Copy link
Member

Description

Add a conditional to the gh-pages.yml github action to exclude dependabot.

Motivation and Context

When merging dependabot PRs to main the publish documentation action is failing because dependabot does not have the correct permissions.

Since dependabot should just be dependency updates, no need to re-publish docs for these PRs anyway. This saves having CI fail every time we merge one a dependabot PR.

Similar fix here: ThePalaceProject/web-opds-client#65

How Has This Been Tested?

Checklist:

  • I have updated the documentation accordingly.
  • All new and existing tests passed.

@jonathangreen jonathangreen requested a review from tdilauro April 17, 2025 13:47
Copy link
Contributor

@tdilauro tdilauro left a comment

Choose a reason for hiding this comment

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

👍🏽

@jonathangreen jonathangreen merged commit 14cc1cb into main Apr 18, 2025
1 check passed
@jonathangreen jonathangreen deleted the chore/ignore-dependabot-gh-pages branch April 18, 2025 14:31
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.

3 participants