Skip to content

Fix typescript errors, finetune mix alias#6138

Merged
apata merged 8 commits intoe2e-feature-tests-reduxfrom
e2e-feature-tests-redux--fix-types
Mar 8, 2026
Merged

Fix typescript errors, finetune mix alias#6138
apata merged 8 commits intoe2e-feature-tests-reduxfrom
e2e-feature-tests-redux--fix-types

Conversation

@apata
Copy link
Copy Markdown
Contributor

@apata apata commented Mar 7, 2026

Changes

Against #6077

  • Fixes Typescript issues
  • Adds Typescript typecheck to CI checks
  • Removes duplication from mix aliases
  • Reformats
  • Adds formatting check to CI checks

Tests

  • This PR does not require tests

Changelog

  • This PR does not make a user-facing change

Documentation

  • This change does not need a documentation update

Dark mode

  • This PR does not change the UI

@apata apata mentioned this pull request Mar 7, 2026
1 task
Comment on lines +212 to +217
- name: Check format
run: npm --prefix ./e2e run check-format

- name: Check types
run: npm --prefix ./e2e run typecheck

Copy link
Copy Markdown
Contributor Author

@apata apata Mar 7, 2026

Choose a reason for hiding this comment

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

Runs once in every shard, so it's a bit wasteful, but I really want these to be enforced

Copy link
Copy Markdown
Contributor

@zoldar zoldar left a comment

Choose a reason for hiding this comment

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

Awesome. Thanks for taking time ironing out the typing ❤️ And the mix task accepting arguments is definitely a better option, thanks for that as well.

@apata apata merged commit 82ab9c5 into e2e-feature-tests-redux Mar 8, 2026
24 checks passed
@apata apata deleted the e2e-feature-tests-redux--fix-types branch March 8, 2026 07:36
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