8000 chore: align build dependencies between acc test and release pipeline by tng1-equinix · Pull Request #351 · equinix/pulumi-equinix · GitHub
[go: up one dir, main page]

Skip to content

chore: align build dependencies between acc test and release pipeline - #351

Merged
tng1-equinix merged 1 commit into
mainfrom
align-deps
Sep 25, 2026
Merged

tng1-equinix merged 1 commit into
mainfrom
align-deps

Conversation

@tng1-equinix
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

Does the PR have any schema changes?

Does the PR have any schema changes?

Generated by schema-tools v0.8.1.

Looking good! No breaking changes found.
No new resources/functions/types.

Maintainer note: consult the runbook for dealing with any breaking changes.

@tng1-equinix tng1-equinix changed the title Align build dependencies between acc test and release pipeline chore: align build dependencies between acc test and release pipeline Sep 23, 2026
@tng1-equinix
tng1-equinix force-pushed the align-deps branch 2 times, most recently from f95ac16 to 450f6b0 Compare September 23, 2026 23:47
NODEVERSION: 20.x
PYTHONVERSION: "3.13.12"
NODEVERSION: 22.x
PYTHONVERSION: "3.9"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why our release pipeline is using 3.13 since it will fail when trying to use setuptools which is removed in 3.12.

3.9 is what we're using in our tests and its working

distribution: temurin
java-version: ${{ env.JAVAVERSION }}
- name: Setup Gradle
uses: gradle/gradle-build-action@v3
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gradle/gradle-build-action is deprecated

uses: actions/setup-node@v7
with:
node-version: ${{ env.NODEVERSION }}
registry-url: https://registry.npmjs.org
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need this because we're not publishing in the tests

@tng1-equinix
tng1-equinix merged commit a57b545 into main Sep 25, 2026
16 checks passed
@tng1-equinix
tng1-equinix deleted the align-deps branch September 25, 2026 17:08
@github-actions
Copy link
Copy Markdown

This PR is included in version 0.32.1 🎉

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.

2 participants

0