Bump deps and use npm7

[skjnldsv]

Please test on your respective apps and see what's breaking ⚠️
(list built automatically, feel free to remove unrelated entries)

1. Remove all devDeps like babel, eslint, webpack, loaders... etc
2. Delete your lockfile
3. Make sure you use npm 7 ⚠️
4. npm i -D @nextcloud/webpack-vue-config@latest @nextcloud/eslint-config@latest @nextcloud/babel-config@latest @nextcloud/browserslist-config@latest @nextcloud/stylelint-config@latest
5. Check your stylelint config https://github.com/nextcloud/stylelint-config#usage
6. Check your babel config https://github.com/nextcloud/babel-config#usage
7. Check your eslint config https://github.com/nextcloud/eslint-config#usage
8. Build and check for errors
9. Profit

---

• activity Use npm 7 activity#593
• announcementcenter Move to node 14 npm 7 announcementcenter#360
• app-tutorial Bump deps and use npm7 #319
• approval
• bookmarks
• browser_warning
• bruteforcesettings
• calendar Bump deps / @nextcloud/webpack-vue-config and use npm7 calendar#3215
• cdav-library
• contacts Bump deps and use npm7 contacts#2281
• cookbook
• deck Move to node 14 and npm 7 deck#3137
• external
• files_automatedtagging
• files_pdfviewer Bump deps and use npm7 files_pdfviewer#432
• files_texteditor
• files_zip
• firstrunwizard
• flow_notifications
• flow_webhooks
• forms Bump Node/Npm forms#987
• groupfolders
• guests
• integration_discourse
• integration_github
• integration_gitlab
• integration_google
• integration_jira
• integration_mastodon
• integration_moodle
• integration_reddit
• integration_twitter
• integration_whiteboard
• integration_zammad
• mail Bump deps and use npm7 mail#5182
• maps
• nextcloud-password-confirmation
• nextcloud-register
• nextcloud-vue Blocked by webpack 4
• notes use npm7, @nextcloud/stylelint-config and update notes#727
• notifications
• officeonline
• password_policy
• photos Bump deps and use npm7 photos#801
• polls
• privacy
• push
• registration Bump node 14 and npm 7 registration#309
• richdocuments
• server Will be the exception until Nextcloud 23 for safety reasons
• sharelisting
• social
• spreed Techdebt/noid/node14 npm7 spreed#5773
• tasks Use npm v7 and simplify overall configuration tasks#1638
• terms_of_service
• text
• user_oidc Move to npm7 and update actions user_oidc#286
• user_retention Bump Node14 npm7 user_retention#361
• viewer Bump deps and use npm7 viewer#955
• workflow_pdf_converter
• vue-dashboard

[raimund-schluessler]

Configuration works fine for Tasks.

[korelstar]

🎉
Works for notes, see nextcloud/notes#727

[jotoeri]

LGTM nextcloud/forms#987

[korelstar]

Just for information: if you use node v15, you do not need to explicitly install npm v7, since it's already included. This simplifies the GitHub workflow a little bit.

[skjnldsv]

Just for information: if you use node v15, you do not need to explicitly install npm v7, since it's already included. This simplifies the GitHub workflow a little bit.

Node lts is still 14 :)
https://nodejs.org/en/about/releases/

[skjnldsv]

One issue I encountered!
If your vue is set to a different version than the vue-template-compiler of the @nextcloud/webpack-vue-config, it will throw (well, I guess you will receive the update or will definitely see the issue). That's the only issue I found so far. But that's an issue with the whole process, it can already happen

[nickvergessen]

announcementcenter works: nextcloud/announcementcenter#360

[nickvergessen]

user retention works: nextcloud/user_retention#361

[raimund-schluessler]

@skjnldsv I think nextcloud-vue is does not have a PR for npm7 yet, right (just asking because it's checked in the list)?

[skjnldsv]

I think nextcloud-vue is does not have a PR for npm7 yet, right (just asking because it's checked in the list)?

Ah sorry, I thought nextcloud-libraries/nextcloud-vue#2037 was it, indeed! 👍
Wanna do it there?

[raimund-schluessler]

I think nextcloud-vue is does not have a PR for npm7 yet, right (just asking because it's checked in the list)?

Ah sorry, I thought nextcloud/nextcloud-vue#2037 was it, indeed! 👍
Wanna do it there?

Yes, I was about to try that yesterday already, but thought it would be cleaner to first merge the common webpack config 😃

[nickvergessen]

Registration works: nextcloud/registration#309

[skjnldsv]

Ok, let's go then, the process seems to be going well?

[skjnldsv]

@korelstar @raimund-schluessler @jotoeri @juliushaertl @artonge @eneiluj @ChristophWurst
You can adjust the package.json engine to also specify which npm version is needed:

	"engines": {
		"node": ">=14.0.0",
		"npm": ">=7.0.0"
	},

[mejo-]

Collectives app also migrated now. Thanks a lot for the detailed instructions @skjnldsv!

Since in our case some extra steps were necessary (in order to make the CI work again), here's the link to our merge request for future reference: https://gitlab.com/collectivecloud/collectives/-/merge_requests/301