Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Docs preview does not work with dependabot PRs #746

Closed
1 task
AetherUnbound opened this issue Jun 2, 2022 · 0 comments · Fixed by #747
Closed
1 task

Docs preview does not work with dependabot PRs #746

AetherUnbound opened this issue Jun 2, 2022 · 0 comments · Fixed by #747
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents

Comments

@AetherUnbound
Copy link
Contributor

Description

Presently the docs preview step is failing on all our dependabot PRs (see #731, #733, #734, #735, #736, #737, #738, #739, #745).

This is because, even though dependabot PRs do not come from a fork, they are explicitly run with read-only permissions (for security reasons I imagine).

We will need to add a check to exclude these PRs from the docs preview workflow.

Additional context

Resolution

  • 🙋 I would be interested in resolving this bug.
@AetherUnbound AetherUnbound added 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents 🤖 aspect: dx Concerns developers' experience with the codebase labels Jun 2, 2022
@krysal krysal mentioned this issue Jun 3, 2022
Merged
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🤖 aspect: dx Concerns developers' experience with the codebase 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Prevent dependabot PRs from running a docs preview WordPress/openverse-api
1 participant
@AetherUnbound