Skip to content

Ignore bikeshed errors for merge base#178

Merged
tobie merged 1 commit into
tobie:masterfrom
Rob--W:ignore-bs-error-in-base
Jun 5, 2026
Merged

Ignore bikeshed errors for merge base#178
tobie merged 1 commit into
tobie:masterfrom
Rob--W:ignore-bs-error-in-base

Conversation

@Rob--W

@Rob--W Rob--W commented May 31, 2026

Copy link
Copy Markdown
Contributor

Fixes #177

@Rob--W

Rob--W commented May 31, 2026

Copy link
Copy Markdown
Contributor Author

Test is failing because I forgot to fix the test expectation after refactoring to use the fixture. I'll fix that up and update the PR.

@Rob--W Rob--W force-pushed the ignore-bs-error-in-base branch from 33b9628 to 171d154 Compare May 31, 2026 14:40
@Rob--W Rob--W force-pushed the ignore-bs-error-in-base branch from 171d154 to 927edb2 Compare May 31, 2026 17:06
@annevk

annevk commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

@tobie what do you think?

@tobie

tobie commented Jun 5, 2026

Copy link
Copy Markdown
Owner

LGTM. Missed that the tests had been fixed.

@tobie tobie merged commit 5afbbb2 into tobie:master Jun 5, 2026
1 check passed
@Rob--W

Rob--W commented Jun 5, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for merging! What do I need to do to trigger the updated/latest version of pr-preview in w3c/webextensions#905 ?

@tobie

tobie commented Jun 5, 2026

Copy link
Copy Markdown
Owner

@Rob--W Just modifying the body of the PR on GH is enough to retrigger a build. You don't even need to push a change.

@tobie

tobie commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Looks like it worked! Congrats and thanks for the fix!

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.

PR Preview should override Bikeshed "Die On" for before document

3 participants