Skip to content

Conversation

rschristian
Copy link
Member

No description provided.

Comment on lines -22 to +21
"version": "latest"
"version": "16.8"
Copy link
Member Author

Choose a reason for hiding this comment

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

ESLint throws a bunch of warnings otherwise

"build": "microbundle -f cjs,es,umd",
"lint": "eslint src/*.{js,jsx}",
"test": "wtr src/*.test.{js,jsx}",
"test:browsers": "wtr src/*.test.{js,jsx} --playwright --browsers chromium firefox webkit",
Copy link
Member Author

@rschristian rschristian Sep 2, 2025

Choose a reason for hiding this comment

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

Playwright is slow to install and rather unwieldy, I don't think it's really needed here anyhow? Our tests aren't that interesting or on the bleeding edge. running in just chromium should be more than fine.

Edit: And, of course, these don't run at the moment anyhow. I gave it a shot but it fell over and I'm not sure it's worth fixing (or at least I won't bother).

@rschristian rschristian marked this pull request as ready for review September 2, 2025 23:36
@rschristian
Copy link
Member Author

rschristian commented Sep 2, 2025

I'll have to update the CI config once approved; giving the CI job a name (which, IMO, makes it look nicer and allows for better expansion later) breaks the check.

@rschristian rschristian force-pushed the chore/bump-deps-and-ci-actions branch from 4316b0b to 847b99e Compare September 3, 2025 00:03
@rschristian rschristian merged commit 4a5346a into master Sep 3, 2025
1 check passed
@rschristian rschristian deleted the chore/bump-deps-and-ci-actions branch September 3, 2025 15:21
@rschristian rschristian mentioned this pull request Sep 6, 2025
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