From b0b165b9a57abc16174fc56118745cef053e445d Mon Sep 17 00:00:00 2001 From: Himanshu Mishra Date: Tue, 3 Aug 2021 10:50:56 +0200 Subject: [PATCH 01/12] create-app: add a prettier:check script in the template Co-authored-by: Mike Lewis Signed-off-by: Himanshu Mishra --- packages/create-app/templates/default-app/package.json.hbs | 1 + 1 file changed, 1 insertion(+) diff --git a/packages/create-app/templates/default-app/package.json.hbs b/packages/create-app/templates/default-app/package.json.hbs index 7ae86c0fbc..a763b3cabb 100644 --- a/packages/create-app/templates/default-app/package.json.hbs +++ b/packages/create-app/templates/default-app/package.json.hbs @@ -19,6 +19,7 @@ "test:all": "lerna run test -- --coverage", "lint": "lerna run lint --since origin/master --", "lint:all": "lerna run lint --", + "prettier:check": "prettier --check .", "create-plugin": "backstage-cli create-plugin --scope internal --no-private", "remove-plugin": "backstage-cli remove-plugin" }, From e7d9f68678dee468b0089bf3bff03734e1989322 Mon Sep 17 00:00:00 2001 From: Himanshu Mishra Date: Tue, 3 Aug 2021 10:51:55 +0200 Subject: [PATCH 02/12] e2e: run prettier check for newly created app Co-authored-by: Mike Lewis Signed-off-by: Himanshu Mishra --- packages/e2e-test/src/commands/run.ts | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/packages/e2e-test/src/commands/run.ts b/packages/e2e-test/src/commands/run.ts index c205b4f527..e4072872d1 100644 --- a/packages/e2e-test/src/commands/run.ts +++ b/packages/e2e-test/src/commands/run.ts @@ -232,7 +232,14 @@ async function createApp( print('Test app created'); - for (const cmd of ['install', 'tsc', 'build', 'lint:all', 'test:all']) { + for (const cmd of [ + 'install', + 'tsc', + 'build', + 'lint:all', + 'prettier:check', + 'test:all', + ]) { print(`Running 'yarn ${cmd}' in newly created app`); await runPlain(['yarn', cmd], { cwd: appDir }); } From 58cdab3d601fa06307ff8eabd11a87984e892b25 Mon Sep 17 00:00:00 2001 From: Himanshu Mishra Date: Tue, 3 Aug 2021 11:11:09 +0200 Subject: [PATCH 03/12] create-app: upgrade prettier version since we are using 2.x in backstage/backstage Co-authored-by: Mike Lewis Signed-off-by: Himanshu Mishra --- packages/create-app/templates/default-app/package.json.hbs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/create-app/templates/default-app/package.json.hbs b/packages/create-app/templates/default-app/package.json.hbs index a763b3cabb..3f913102cb 100644 --- a/packages/create-app/templates/default-app/package.json.hbs +++ b/packages/create-app/templates/default-app/package.json.hbs @@ -38,7 +38,7 @@ "@spotify/prettier-config": "^7.0.0", "concurrently": "^6.0.0", "lerna": "^4.0.0", - "prettier": "^1.19.1" + "prettier": "^2.3.2" }, "prettier": "@spotify/prettier-config", "lint-staged": { From 94abf07f1ddd0381bed5e6a4ad95b82327f5dc39 Mon Sep 17 00:00:00 2001 From: Himanshu Mishra Date: Tue, 3 Aug 2021 11:12:14 +0200 Subject: [PATCH 04/12] fix prettier issues in the create-app default template Co-authored-by: Mike Lewis Signed-off-by: Himanshu Mishra --- packages/create-app/templates/default-app/README.md | 2 +- .../templates/default-app/catalog-info.yaml.hbs | 4 ++-- .../default-app/packages/app/public/index.html | 2 +- .../templates/default-app/packages/app/src/App.tsx | 10 +++++----- .../templates/default-app/packages/app/src/apis.ts | 9 +++++++-- .../packages/app/src/components/catalog/EntityPage.tsx | 2 +- .../packages/backend/src/plugins/catalog.ts | 6 ++++-- .../packages/backend/src/plugins/scaffolder.ts | 4 +--- .../default-app/packages/backend/src/types.ts | 2 +- 9 files changed, 23 insertions(+), 18 deletions(-) diff --git a/packages/create-app/templates/default-app/README.md b/packages/create-app/templates/default-app/README.md index 07cf6fd136..8c7c4373fe 100644 --- a/packages/create-app/templates/default-app/README.md +++ b/packages/create-app/templates/default-app/README.md @@ -7,4 +7,4 @@ To start the app, run: ```sh yarn install yarn dev -``` \ No newline at end of file +``` diff --git a/packages/create-app/templates/default-app/catalog-info.yaml.hbs b/packages/create-app/templates/default-app/catalog-info.yaml.hbs index 265864e235..e01ce8f756 100644 --- a/packages/create-app/templates/default-app/catalog-info.yaml.hbs +++ b/packages/create-app/templates/default-app/catalog-info.yaml.hbs @@ -5,8 +5,8 @@ metadata: description: An example of a Backstage application. # Example for optional annotations # annotations: - # github.com/project-slug: backstage/backstage - # backstage.io/techdocs-ref: dir:. + # github.com/project-slug: backstage/backstage + # backstage.io/techdocs-ref: dir:. spec: type: website owner: john@example.com diff --git a/packages/create-app/templates/default-app/packages/app/public/index.html b/packages/create-app/templates/default-app/packages/app/public/index.html index 5653173480..b93896c6da 100644 --- a/packages/create-app/templates/default-app/packages/app/public/index.html +++ b/packages/create-app/templates/default-app/packages/app/public/index.html @@ -65,7 +65,7 @@ <% } %> - +