Skip to content

chore: Update tmp#319

Merged
dylanlerch merged 1 commit into
mainfrom
dylan/update-tmp
Jun 19, 2026
Merged

chore: Update tmp#319
dylanlerch merged 1 commit into
mainfrom
dylan/update-tmp

Conversation

@dylanlerch

@dylanlerch dylanlerch commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Part of DEVEX-449

@dylanlerch dylanlerch requested a review from a team as a code owner June 16, 2026 22:59
Comment thread package.json
"@octopusdeploy/api-client": "3.7.0",
"axios": "^1.17.0",
"tmp": "0.2.6"
"tmp": "0.2.7"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same question as OctopusDeploy/create-release-action#698 do we need a new version of the action?

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.

Considering this didn't change the index.js, I don't think so. Possible this should be a dev dependency?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah probably should be, do we know where its used?

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.

I'll do some dig dig

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.

Dropped esbuild (it was wrong), and just doing tmp now. No changes to the dist folder, so happy with this as a chore. Going to leave it where it is (in dependencies) for now.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@dylanlerch dylanlerch merged commit 5914421 into main Jun 19, 2026
6 checks passed
@dylanlerch dylanlerch deleted the dylan/update-tmp branch June 19, 2026 03:51
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