* feat: npm -> pnpm
This resolves the issues with the gatsby client (gatsby-plugin-pnpm
deals with the fact that gatsby is relying on its own dependencies
being de-duped)
and challenge-editor (which doesn't seem to want to automatically install
codemirror and needed its own eslint config)
* fix: correct mocha path for curriculum tests
* fix: use select workspace with -F not -w
* fix: reorganise packages and restrict hoisting
pnpm works best if the workspaces keep their own dependencies, since
dependencies are not flattened and then what node resolves from a
require is predictable.
@types seem to be a special case and more care is required to prevent
them getting smushed together in the root (hence the .npmrc)
* fix: add types for tools + root
* fix: decouple challenge-auditor from client
* fix: add ui-components types
* fix(client): use the latest types for react 16
* fix: prettify
* fix: prettierignore pnpm-lock
* fix: relax hoisting
Turns out pnpm works just fine with types. I don't know what was going
wrong before, but there are no-longer any type conflicts.
* fix: add @redux-saga/core to fix eslint issue
It seems to only be redux-saga that import/named can't cope with, so it
is probably okay to work around this one.
* chore: add chai to tools/scripts/build
* fix: add store to root for cypress
* fix: allow cypress to download binaries
If we want to keep preventing cypress from downloading binaries, we can
figure out a workaround, but I'm allowing it to ease the transition to
pnpm.
My guess about why this is happening is that npm triggers Cypress's
postinstall script, but pnpm does not (because pnpm install only
installs if necessary, perferring to link)
* chore: re-enable pre/post scripts
* fix: update build scripts for client
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
* chore: update engines to use pnpm
* fix: enable choice of (super)block for tests
Only 'nix machines for now.
* chore: pin pnpm to version 7
* chore: remove last npms
Except web + curriculum-server. I'll update them when I start work on
them again.
* fix: lockfile check to catch any package-locks
* fix(action): install pnpm for upcoming tests
* chore: add nodemon to new api
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
react-monaco-editor needed rolling back due to an incompatibility with
react 16
monaco-editor then needed rolling back and the editor config, too.
Some @types needed to be added, some lint deps needed updating.
* feat(ts-migrate): rename sass-compile.js to ts
* feat(ts-migrate): rename test-evaluator.js to ts
* feat(ts-migrate): add webworker in client tsconfig
* fix(ts-migrate): fix errors in sass-compile.ts
* chore(ts-migrate): install chai types in client
* fix(ts-migrate): fix errors in test-evaluator.ts
* fix(ts-migrate): rename extensions in webpack-worker
* fix(ts-migrate): separate tsconfig for workers
* fix(ts-migrate): add worker tsconfig to parser options
* chore(ts-migrate): remove unnecessary comment
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* fix(ts-migrate): use let instead of const
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* fix(ts-migrate): fix eslint errors in sass-compile.ts
* fix(ts-migrate): fix eslint errors in test-evaluator.ts
* chore(ts-migrate): use unknown instead of generics
* chore(ts-migrate): revert worker tsconfig
* chore(ts-migrate): add libs in client tsconfig
* fix(ts-migrate): use ctx alias in test-evaluator.ts
* fix(ts-migrate): use ctx alias in sass-compile.ts
* chore: fix errors
Co-authored-by: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
* fix: remove circular dependency
redux depended on templates/Challenges/redux and vice versa. This
meant that import order mattered and confusing bugs could arise.
(cherry picked from commit 7d67a4e70922bbb3051f2f9982dcc69e240d43dc)
* feat: require imports to be in alphabetical order
Import order generally does not matter, but there are edge cases
(circular imports and css imports, for example) where changing order
changes behaviour
(cherry picked from commit b8d1393a91ec6e068caf8e8498a5c95df68c2b2c)
* chore: order imports
* fix: lift up challenge description + title comps
This brings the classic Show closer to the others as they
now all create the description and title components
* fix: remove donation-saga/index circular import
(cherry picked from commit 51a44ca668a700786d2744feffeae4fdba5fd207)
* refactor: extract action-types from settings
(cherry picked from commit 25e26124d691c84a0d0827d41dafb761c686fadd)
* fix: lint errors
* feat: prevent useless renames
* feat: use webpack caching in development
This cuts build time for build-workers considerably.
* fix: update gitpod config
Co-authored-by: Shaun Hamilton <shauhami020@gmail.com>
* fix(client): add cache-busting hashes to chunks
* fix: create cache-busting names for the workers
Prior to this PR the first webpack compilation gave the workers static
names. This can cause caching problems, so this PR adds hashes to
their names to invalidate the cache.
In order for Gatsby to find them, the names are added to the
config directory.
