Merge pull request #86 from owncloud/chore/linter-dependencies-depend…

…abot-codeowners Update linting, upgrade dependencies, add dependabot, update codeowners
owncloud · Oct 12, 2021 · 567cc50 · 567cc50
2 parents 2ce6b5c + 83c8033
commit 567cc50
Show file tree

Hide file tree

Showing 26 changed files with 1,888 additions and 1,821 deletions.
diff --git a/.drone.star b/.drone.star
@@ -56,23 +56,20 @@ def changelog(ctx):
       {
         'name': 'generate',
         'image': 'toolhippie/calens:latest',
-        'pull': 'always',
         'commands': [
           'calens >| CHANGELOG.md',
         ],
       },
       {
         'name': 'diff',
         'image': 'owncloudci/alpine:latest',
-        'pull': 'always',
         'commands': [
           'git diff',
         ],
       },
       {
         'name': 'output',
         'image': 'owncloudci/alpine:latest',
-        'pull': 'always',
         'commands': [
           'cat CHANGELOG.md',
         ],
@@ -246,15 +243,13 @@ def tests(ctx):
       {
         'name': 'install',
         'image': 'owncloudci/nodejs:14',
-        'pull': 'always',
         'commands': [
           'yarn install --frozen-lockfile',
         ],
       },
       {
         'name': 'lint',
         'image': 'owncloudci/nodejs:14',
-        'pull': 'always',
         'commands': [
           'yarn lint',
         ],
@@ -263,7 +258,6 @@ def tests(ctx):
       {
         'name': 'build',
         'image': 'owncloudci/nodejs:14',
-        'pull': 'always',
         'commands': [
           'yarn build',
         ],
@@ -272,7 +266,6 @@ def tests(ctx):
       {
         'name': 'unit',
         'image': 'owncloudci/nodejs:14',
-        'pull': 'always',
         'commands': [
           'yarn test:unit'
         ],
@@ -281,7 +274,6 @@ def tests(ctx):
       {
         'name': 'integration',
         'image': 'owncloudci/nodejs:14',
-        'pull': 'always',
         'commands': [
           'yarn test:integration'
         ],
@@ -311,7 +303,6 @@ def release(ctx):
       {
         'name': 'build',
         'image': 'owncloudci/nodejs:14',
-        'pull': 'always',
         'commands': [
           'yarn install --frozen-lockfile',
           'yarn build'
@@ -325,7 +316,6 @@ def release(ctx):
       {
         'name': 'changelog',
         'image': 'toolhippie/calens:latest',
-        'pull': 'always',
         'commands': [
           'calens --version %s -o dist/CHANGELOG.md' % ctx.build.ref.replace("refs/tags/v", "").split("-")[0],
         ],
@@ -338,7 +328,6 @@ def release(ctx):
       {
         'name': 'publish-github',
         'image': 'plugins/github-release:latest',
-        'pull': 'always',
         'settings': {
           'api_key': {
             'from_secret': 'github_token',
@@ -356,7 +345,6 @@ def release(ctx):
       {
         'name': 'publish-npm',
         'image': 'plugins/npm:latest',
-        'pull': 'always',
         'settings': {
           'username': {
             'from_secret': 'npm_username',

diff --git a/.eslintrc.js b/.eslintrc.js
@@ -5,17 +5,21 @@ module.exports = {
   },
   extends: ['plugin:vue/recommended', 'eslint:recommended', '@vue/prettier'],
   parserOptions: {
-    parser: "@babel/eslint-parser",
+    parser: '@babel/eslint-parser',
     ecmaVersion: 6,
-    sourceType: "module"
+    sourceType: 'module'
   },
   rules: {
     'no-console': process.env.NODE_ENV === 'production' ? 'warn' : 'off',
     'no-debugger': process.env.NODE_ENV === 'production' ? 'warn' : 'off'
   },
   overrides: [
     {
-      files: ['**/__tests__/*.{j,t}s?(x)', '**/tests/unit/**/*.spec.{j,t}s?(x)', '**/tests/integration/**/*.spec.{j,t}s?(x)'],
+      files: [
+        '**/__tests__/*.{j,t}s?(x)',
+        '**/tests/unit/**/*.spec.{j,t}s?(x)',
+        '**/tests/integration/**/*.spec.{j,t}s?(x)'
+      ],
       env: {
         jest: true
       }

diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -1 +1 @@
-* LukasHirt
+* @kulmann @pascalwengerter @fschade @paulcod3
diff --git a/.github/dependabot.yml b/.github/dependabot.yml
@@ -0,0 +1,10 @@
+version: 2
+updates:
+- package-ecosystem: npm
+  directory: "/"
+  schedule:
+    interval: weekly
+    day: "monday"
+  labels:
+    - "Status:Needs-Review"
+    - "Topic:Dependencies"
diff --git a/.prettierrc.json b/.prettierrc.json
@@ -1,5 +1,6 @@
 {
   "printWidth": 100,
   "semi": false,
-  "singleQuote": true
-}
+  "singleQuote": true,
+  "trailingComma": "none"
+}
diff --git a/changelog/unreleased/bugfix-update-dependencies b/changelog/unreleased/bugfix-update-dependencies
@@ -0,0 +1,5 @@
+Bugfix: Update dependencies
+
+We've updated some of this project's dependencies.
+
+https://github.com/owncloud/file-picker/pull/86
diff --git a/jest.config.js b/jest.config.js
@@ -11,13 +11,13 @@ module.exports = {
     '<rootDir>/src/**/*.{vue, js}',
     '!<rootDir>/src/main.js',
     '!<rootDir>/src/services/auth.js',
-    '!<rootDir>/src/assets/**/*',
+    '!<rootDir>/src/assets/**/*'
   ],
   coverageDirectory: '<rootDir>/coverage',
   moduleFileExtensions: ['js', 'vue'],
   moduleNameMapper: {
     '^@/(.*)$': '<rootDir>/src/$1',
-    '^.+\\.(css|scss)$': 'babel-jest',
+    '^.+\\.(css|scss)$': 'babel-jest'
   },
   modulePaths: ['<rootDir>'],
   rootDir: path.resolve(__dirname, './'),
@@ -31,7 +31,7 @@ module.exports = {
     '^.+\\.js$': 'babel-jest',
     '^.+\\.vue$': 'vue-jest',
     '^.+\\.(jpg|ico|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$':
-      '<rootDir>/__mocks__/file.js',
+      '<rootDir>/__mocks__/file.js'
   },
-  transformIgnorePatterns: ['node_modules/(?!lodash-es/.*)'],
+  transformIgnorePatterns: ['node_modules/(?!lodash-es/.*)']
 }
diff --git a/package.json b/package.json
@@ -46,13 +46,17 @@
     "oidc-client": "^1.11.5",
     "owncloud-design-system": "^7.2.0",
     "owncloud-sdk": "^1.0.0-2296",
+    "postcss-import": "^12.0.1",
+    "postcss-url": "^9.0.0",    
     "tippy.js": "^6.3.1",
     "uikit": "3.5.16",
     "vue-avatar": "^2.3.3",
     "vue-datetime": "^1.0.0-beta.14",
     "vue-gettext": "^2.1.12",
     "vue-inline-svg": "^2.0.0",
-    "vue-select": "^3.11.2"
+    "vue-select": "^3.11.2",
+    "webfontloader": "^1.6.28",
+    "weekstart": "^1.0.0"
   },
   "devDependencies": {
     "@babel/core": "^7.13.16",
@@ -81,6 +85,8 @@
     "vue": "^2.6.12",
     "vue-jest": "^3.0.7",
     "vue-loader": "^15.9.6",
-    "vue-template-compiler": "^2.6.12"
+    "vue-template-compiler": "^2.6.12",
+    "webpack": "^4.0.0"
   }
 }
+
diff --git a/src/App.vue b/src/App.vue
@@ -53,15 +53,15 @@ if (!Vue.prototype.$gettext) {
     cs: 'Czech',
     fr: 'Français',
     it: 'Italiano',
-    gl: 'Galego',
+    gl: 'Galego'
   }
   const translations = merge({}, filePickerTranslations, odsTranslations)
 
   Vue.use(VueGettext, {
     availableLanguages: supportedLanguages,
     defaultLanguage: navigator.language.substring(0, 2),
     translations,
-    silent: true,
+    silent: true
   })
 }
 
@@ -70,77 +70,77 @@ export default {
 
   components: {
     FilePicker,
-    Login,
+    Login
   },
 
   props: {
     variation: {
       type: String,
       required: true,
-      validator: (value) => value === 'resource' || value === 'location',
+      validator: (value) => value === 'resource' || value === 'location'
     },
     configLocation: {
       type: String,
       required: false,
-      default: () => window.location.origin + '/file-picker-config.json',
+      default: () => window.location.origin + '/file-picker-config.json'
     },
     bearerToken: {
       type: String,
       required: false,
-      default: null,
+      default: null
     },
     configObject: {
       type: [Object, String],
       required: false,
-      default: null,
+      default: null
     },
     isSdkProvided: {
       type: Boolean,
       required: false,
-      default: false,
+      default: false
     },
     selectBtnLabel: {
       type: String,
       required: false,
-      default: null,
+      default: null
     },
     isSelectBtnDisplayed: {
       type: Boolean,
       required: false,
-      default: true,
+      default: true
     },
     cancelBtnLabel: {
       type: String,
       required: false,
-      default: null,
+      default: null
     },
     isOdsProvided: {
       type: Boolean,
       required: false,
-      default: false,
+      default: false
     },
     locale: {
       type: String,
       required: false,
-      default: null,
+      default: null
     },
     isInitialFocusEnabled: {
       type: Boolean,
       required: false,
-      default: false,
-    },
+      default: false
+    }
   },
 
   data: () => ({
     authInstance: null,
     state: 'loading',
-    config: null,
+    config: null
   }),
 
   computed: {
     currentLocale() {
       return this.locale || navigator.language.substring(0, 2)
-    },
+    }
   },
 
   created() {
@@ -169,11 +169,11 @@ export default {
         this.$client.init({
           baseUrl: this.config.server,
           auth: {
-            bearer: bearerToken,
+            bearer: bearerToken
           },
           headers: {
-            'X-Requested-With': 'XMLHttpRequest',
-          },
+            'X-Requested-With': 'XMLHttpRequest'
+          }
         })
       }
 
@@ -226,8 +226,8 @@ export default {
 
     onFolderLoaded(folder) {
       this.$emit('folderLoaded', folder)
-    },
-  },
+    }
+  }
 }
 </script>