Migrate to npm 7

[skjnldsv]

Yet another shot at it.
Lots of outdated sections 😩

I didn't change the babel config as this is causing export issues and make everything not defined: dav is not defined OC is not defined etc etc. Most likely an issue with umd/commonjs exports

---

Followups:

• Address the vue/no-mutating-props
  

  server/.eslintrc.js

  Line 18 in 7ca81f3

  'vue/no-mutating-props': 'warn',

• Migrate from jsunit to jest

[skjnldsv]

NODB failure is unrelated 🎉

[juliusknorr]

🥳

[szaimen]

Did you push the production or dev js?

[skjnldsv]

Did you push the production or dev js?

[szaimen]

Did you push the production or dev js?

So this checks also if the correct js was pushed?

[skjnldsv]

So this checks also if the correct js was pushed?

Yes, always have been 🔫 😉

server/.github/workflows/node.yml

Lines 44 to 52 in 717636f

	- name: Check webpack build changes
	run: |
	bash -c "[[ ! \"`git status --porcelain `\" ]] || exit 1"
	- name: Show changes on failure
	if: failure()
	run: |
	git status
	git --no-pager diff

[szaimen]

okay and shouldn't we wait until the tests finish?

[skjnldsv]

okay and shouldn't we wait until the tests finish?

Finished! 🛎️ 😁

[artonge]

Did you push the production or dev js?

So this checks also if the correct js was pushed?

Yes. It compiles in production mode, then checks if nothing as changed

See comments