GLSP-1578: Minor follow up fixes (#447)

- Ensure that we are also publish if there are changes in the examples
directories (publish-next.yml)
- Add prettier/eslint ignore files
- Clean up root scripts and add scripts for formatting

Author: tortmayr

.eslintignore

@@ -0,0 +1,13 @@
+# Compiled output
+**/dist/
+**/lib/
+**/node_modules/
+
+# Build files
+**/.eslintrc.js
+**/esbuild.js
+
+# Other generated files
+**/*.map
+
+**/scripts/

.eslintrc.js

@@ -2,7 +2,6 @@
 module.exports = {
     root: true,
     extends: '@eclipse-glsp',
-    ignorePatterns: ['**/{node_modules,lib}', '**/.eslintrc.js', '**/scripts/*.ts'],
 
     parserOptions: {
         tsconfigRootDir: __dirname,

.github/workflows/publish-next.yml

@@ -23,10 +23,10 @@ jobs:
           # Required for lerna to determine the version of the next package.
           fetch-depth: 0
           ref: ${{ github.event.workflow_run.head_commit.id || github.sha }}
-      - name: Check for changes in "packages" directory
+      - name: Check for changes in "packages" or "examples" directory
         id: check_changes
         run: |
-          if git diff --name-only HEAD^ HEAD | grep -qE '^packages'; then
+          if git diff --name-only HEAD^ HEAD | grep -qE '^(packages|examples)'; then
             echo "changes=true" >> $GITHUB_ENV
           else
             echo "changes=false" >> $GITHUB_ENV

.prettierignore

@@ -0,0 +1,17 @@
+# Dependencies
+node_modules/
+
+# Build outputs
+lib/
+dist/
+
+# Generated files
+*.min.js
+*.min.css
+
+# Lock files
+package-lock.json
+yarn.lock
+
+# Logs
+*.log

examples/workflow-standalone/webpack.config.js

@@ -73,7 +73,7 @@ module.exports = {
         new webpack.WatchIgnorePlugin({ paths: [/\.js$/, /\.d\.ts$/] }),
         new webpack.DefinePlugin({
             GLSP_SERVER_HOST: JSON.stringify(process.env.GLSP_SERVER_HOST || 'localhost'),
-            GLSP_SERVER_PORT: JSON.stringify(process.env.GLSP_SERVER_PORT || '8081'),
+            GLSP_SERVER_PORT: JSON.stringify(process.env.GLSP_SERVER_PORT || '8081')
         })
     ]
 };

package.json

@@ -11,17 +11,17 @@
     "build": "yarn compile && yarn bundle",
     "bundle": "yarn standalone bundle",
     "check:headers": "glsp checkHeaders . -t lastCommit",
-    "check:pr": "yarn all && yarn check:headers",
+    "check:pr": "yarn all && yarn check:format && yarn check:headers",
     "clean": "lerna run clean && rimraf coverage .nyc_output",
     "compile": "tsc -b",
+    "format": "prettier --write .",
+    "format:check": "prettier --check .",
     "generate:index": "lerna run generate:index && yarn lint:fix",
     "lint": "eslint --ext .ts,.tsx .",
     "lint:ci": "yarn lint --output-file eslint_report.json --format json",
     "lint:fix": " yarn lint --fix",
     "prepare": "yarn build",
-    "publish:latest": "lerna publish from-git --no-git-reset --no-git-tag-version --no-push",
     "publish:next": "lerna publish preminor --exact --canary --preid next --dist-tag next --no-git-reset --no-git-tag-version --no-push --ignore-scripts --yes",
-    "publish:prepare": "lerna version --ignore-scripts --yes --no-push",
     "standalone": "yarn --cwd ./examples/workflow-standalone",
     "start:exampleServer": "yarn standalone start:exampleServer",
     "test": "lerna run test --no-bail",

tsconfig.eslint.json

@@ -3,6 +3,5 @@
   "compilerOptions": {
     "noEmit": true
   },
-  "exclude": ["**/node_modules", "**/.eslintrc.js"],
-  "include": ["packages/*/src", "examples/*/src", "**/workflow-glsp/scripts"]
+  "include": ["packages/*/src", "examples/*/src"]
 }