Skip to content

Conversation

@alxndrsn
Copy link
Contributor

Softer alternative to #532


Closes #531

Related:

Closes #

I have verified this PR works in these browsers (latest versions):

  • Chrome
  • Firefox
  • Safari (macOS)
  • Safari (iOS)
  • Chrome for Android
  • Not applicable

What else has been done to verify that this works as intended?

Why is this the best possible solution? Were any other approaches considered?

How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?

Do we need any specific form for testing your changes? If so, please attach one.

What's changed

@changeset-bot
Copy link

changeset-bot bot commented Oct 30, 2025

⚠️ No Changeset found

Latest commit: 99ceea4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@alxndrsn alxndrsn mentioned this pull request Oct 30, 2025
6 tasks
@alxndrsn alxndrsn marked this pull request as ready for review October 31, 2025 06:27
@alxndrsn alxndrsn requested a review from garethbowen October 31, 2025 06:27
@alxndrsn
Copy link
Contributor Author

@garethbowen this is the alternative approach you suggested at #532 (review)

As you suggest, this does seem like a lower-risk approach.

Copy link
Collaborator

@garethbowen garethbowen left a comment

Choose a reason for hiding this comment

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

Sold.

My next thought is... why isn't this run for every build? It's not like man is going to be useful in GHA. Is it worth doing everywhere to protect against the next package with a long man rebuild?

@alxndrsn alxndrsn merged commit a959fa8 into getodk:main Nov 3, 2025
38 checks passed
@alxndrsn alxndrsn deleted the pw-softer branch November 3, 2025 11:10
@alxndrsn
Copy link
Contributor Author

alxndrsn commented Nov 3, 2025

Is it worth doing everywhere to protect against the next package with a long man rebuild?

I think that would be ideal, and I think that's what actions/runner-images#10977 is asking too 🤷

An alternative approach would be a setup-playwright action which combines:

run: sudo bash -ec 'echo "set man-db/auto-update false" | debconf-communicate; dpkg-reconfigure man-db'

with

run: 'yarn playwright install ${{ matrix.browser }} --with-deps'

Something like:

- uses: actions/setup-playwright
  with:
    browser: ${{ matrix.browser }}

Ideally that action could then be shared across projects in @getodk, but might need to handle yarn vs npm differences, and maybe not worth the effort vs copy+paste+modify for each project.


Microsoft (maintainers of playwright) previously published: https://github.com/microsoft/playwright-github-action, but it's now deprecated.

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.

ci: man-db updates are taking significant build time

2 participants