Skip to content

Commit f0a7169

Browse files
committed
Fix publishing process to include dist directory
My PR #2559 removed a postinstall build step which our publish action depended on. This replaces the missing build step.
1 parent fed24b4 commit f0a7169

3 files changed

Lines changed: 14 additions & 2 deletions

File tree

.changeset/curly-windows-rule.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
---
2+
"@apollo/composition": patch
3+
"apollo-federation-integration-testsuite": patch
4+
"@apollo/gateway": patch
5+
"@apollo/federation-internals": patch
6+
"@apollo/query-graphs": patch
7+
"@apollo/query-planner": patch
8+
"@apollo/subgraph": patch
9+
---
10+
11+
Supersedes v2.4.4 due to a publishing error with no dist/ folder
12+

.github/workflows/release.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ jobs:
3434
with:
3535
title: "release: on branch ${{ github.ref_name }}"
3636
createGithubReleases: true
37-
publish: npm run publish
37+
publish: npm run build-and-publish
3838
# workaround for https://github.com/changesets/action/issues/203, includes an `npm i` after running the version command
3939
version: npm run changeset-version
4040
env:

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
"hints-doc:check": "npm run hints-doc && git diff --exit-code",
2525
"gen-test-supergraph": "ts-node ./query-planner-js/src/__tests__/genTestSupergraph.ts",
2626
"changeset-version": "changeset version && npm i",
27-
"publish": "changeset publish",
27+
"build-and-publish": "npm run compile && changeset publish",
2828
"spell:check": "cspell lint --no-progress --config .cspell/cspell.yml || (echo 'Add any real words to ./cspell/cspell-dict.txt.'; exit 1)",
2929
"use-repo-git-hooks": "git config core.hooksPath .git-hooks"
3030
},

0 commit comments

Comments
 (0)