Skip to content

fix: compile before upgrade in upgrade-main workflow - #284

Open
revmischa wants to merge 1 commit into
mainfrom
fix/upgrade-workflow-compile
Open

revmischa wants to merge 1 commit into
mainfrom
fix/upgrade-workflow-compile

Conversation

@revmischa

Copy link
Copy Markdown
Member

The monthly upgrade workflow has failed every run since at least April. The October log finally shows why:

Error: Expected to find .jsii file in /home/runner/work/cdk-nextjs/cdk-nextjs, but no such file found
Synthesis failed: Task "default" failed when executing "ts-node --project projenrc/tsconfig.json .projenrc.ts"

projen upgrade ends by re-running npx projen, and .projenrc.ts reads the jsii assembly to build the Optional* structs from this package's own types. build.yml and release.yml already patch a npx projen compile && in front of their main step for exactly this reason; upgrade-main.yml never got the same patch. Adds it.

The monthly upgrade workflow has failed every run since at least April.
projen upgrade ends by re-running projen, and .projenrc.ts needs the
compiled jsii assembly to generate the Optional* structs, so synth died
with 'Expected to find .jsii file'. build.yml and release.yml already
patch in a compile step; do the same for upgrade-main.yml.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant