Skip to content

Commit

Permalink
chore(ci): pin version of cssstyle to allow upgrade-main to succeed (#…
Browse files Browse the repository at this point in the history
…469)

Signed-off-by: team-tf-cdk <github-team-tf-cdk@hashicorp.com>
Co-authored-by: team-tf-cdk <github-team-tf-cdk@hashicorp.com>
  • Loading branch information
xiehan and team-tf-cdk authored Jan 14, 2025
1 parent 02886a0 commit f0ce2e8
Show file tree
Hide file tree
Showing 8 changed files with 139 additions and 90 deletions.
6 changes: 5 additions & 1 deletion .github/workflows/upgrade-node.yml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 5 additions & 0 deletions .projen/deps.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions .projenrc.ts
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,8 @@ project.addDevDeps(
"@typescript-eslint/eslint-plugin@^6",
"@typescript-eslint/parser@^6"
);
// This is a temporary workaround to allow upgrade-main to succeed until we upgrade to Node 20
project.package.addPackageResolutions(`cssstyle@4.1.0`);

project.addFields({ publishConfig: { access: "public" } });

Expand Down
3 changes: 3 additions & 0 deletions package.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions projenrc/upgrade-node.ts
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,9 @@ export class UpgradeNode {
"This PR increases the minimum supported Node.js version",
"to `${{ steps.latest_version.outputs.value }}` from `${{ steps.current_version.outputs.value }}`",
"because version ${{ steps.current_version.outputs.short }} is less than 30 days away from EOL.",
"\n\nWhile this PR could be merged as-is, it is recommended that you scan the code (especially `.projenrc.ts`)",
"to see if there are any comments indicating changes that can/should be made when upgrading Node, such as:",
"\n```// The following line can be removed when upgrading to Node ${{ steps.latest_version.outputs.short }}```",
].join(" "),
labels: "automerge,automated,security",
token: "${{ secrets.PROJEN_GITHUB_TOKEN }}",
Expand Down
2 changes: 2 additions & 0 deletions src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -318,6 +318,8 @@ export class CdktfProviderProject extends cdk.JsiiProject {
"@action-validator/core",
"@action-validator/cli"
);
// This is a temporary workaround to allow upgrade-main to succeed until we upgrade to Node 20
this.package.addPackageResolutions(`cssstyle@4.1.0`);

// Default memory is 7GB: https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners#supported-runners-and-hardware-resources
// Custom Runners we use have 32GB of memory
Expand Down
40 changes: 35 additions & 5 deletions test/__snapshots__/index.test.ts.snap

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit f0ce2e8

Please sign in to comment.