style: enforce comma-dangle

The ESLint comma-dangle rule issues warnings but doesn't fail the CI.
Running npm run lint:fix adds missing commas automatically. This often
leads to committing unrelated changes. Now, the CI enforces the dangling
comma rule.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

.eslintrc.js

@@ -22,6 +22,7 @@ module.exports = {
 		'plugin:cypress/recommended',
 	],
 	rules: {
+		'comma-dangle': 'error',
 		'no-tabs': 'warn',
 		// TODO: make sure we fix this as this is bad vue coding style.
 		// Use proper sync modifier

apps/dav/src/components/AbsenceForm.vue

@@ -219,7 +219,7 @@ export default {
 					lastDay: formatDateAsYMD(this.lastDay),
 					status: this.status,
 					message: this.message,
-					replacementUserId: this.replacementUser?.user ?? null
+					replacementUserId: this.replacementUser?.user ?? null,
 				})
 				showSuccess(this.$t('dav', 'Absence saved'))
 			} catch (error) {

apps/dav/src/components/ExampleContentDownloadButton.vue

@@ -26,13 +26,13 @@ export default {
 	name: 'ExampleContentDownloadButton',
 	components: {
 		NcButton,
-		IconDownload
+		IconDownload,
 	},
 	props: {
 		href: {
 			type: String,
 			required: true,
-		}
+		},
 	},
 }
 </script>

apps/dav/src/settings-example-content.js

@@ -10,7 +10,7 @@ Vue.mixin({
 	methods: {
 		t: translate,
 		$t: translate,
-	}
+	},
 })
 
 const View = Vue.extend(ExampleContentSettingsSection);

apps/dav/src/views/ExampleContentSettingsSection.vue

@@ -33,6 +33,6 @@ export default {
 		hasCalendarApp() {
 			return loadState('dav', 'calendarEnabled')
 		},
-	}
+	},
 }
 </script>