Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(refactor): restructure repo & support WDIO v9 #623

Merged
merged 125 commits into from
Jul 16, 2024
Merged

Conversation

goosewobbler
Copy link
Member

@goosewobbler goosewobbler commented Jun 13, 2024

Restructuring the repo into a proper monorepo using Turborepo and PNPM workspaces. This includes the update to support WDIO v9.

Note that we are using npm to build the forge examples. This is obviously very hacky but we should be able to make it a lot cleaner (and faster) if / when Forge supports PNPM.

Closes #443.

TODO:

  • Ensure builds work
  • Ensure tests pass
  • Ensure tests pass on CI
  • Ensure linting works
  • Extract utils and types into packages
  • Ensure Mac universal tests work
  • Fix Forge mac-universal E2E failures #576
  • Fix Windows CI b0rkage on example-forge-esm:build
  • Work around Windows CI e2es taking forever (Downgrade Electron)
  • Fix Windows CI EPERM errors on forge builds
  • Fix Linux CI DevToolsActivePort b0rkage
  • Streamline CJS build approach
  • Investigate EPERM errors with Forge v7 on Windows CI #635
  • Update units
  • Dev tasks for watching files and building dependent packages on save
  • Optimise scripts
  • Clean up
  • Docs - build info & graphs
  • Docs - any other updates
  • Support WDIO v9
  • Update release action & scripts

@goosewobbler goosewobbler changed the title Restructure Repo (refactor): restructure repo Jun 13, 2024
@goosewobbler goosewobbler force-pushed the sm/restructure-repo branch 2 times, most recently from 937eac3 to f1bf735 Compare June 13, 2024 23:37
@goosewobbler goosewobbler added the WIP Work in progress label Jun 14, 2024
@goosewobbler goosewobbler linked an issue Jul 10, 2024 that may be closed by this pull request
7 tasks
@goosewobbler goosewobbler changed the title (refactor): restructure repo (refactor): restructure repo & support WDIO v9 Jul 11, 2024
@goosewobbler
Copy link
Member Author

goosewobbler commented Jul 11, 2024

@christian-bromann I think this is good to go. Once merged I'll release some test builds. Tracking the remaining release tasks in #533.

Copy link
Contributor

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

Fantastic work 👍

I don't see anything critical here. One questions though: do we want to keep the graph images in the repo or are they part of dev documentation?

@goosewobbler
Copy link
Member Author

do we want to keep the graph images in the repo or are they part of dev documentation?

Good catch, they should go in the docs.

@goosewobbler goosewobbler merged commit d6dbaf3 into main Jul 16, 2024
4 checks passed
@goosewobbler goosewobbler deleted the sm/restructure-repo branch July 16, 2024 14:46
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.

Forge mac-universal E2E failures Update service for WDIO v9 Restructure Repo
2 participants