Skip to content

fix: require is_secret attribute on deploy environment variables#621

Merged
ndhoule merged 1 commit intomasterfrom
fix/deploy-variables-require-is-secret-attr
Feb 18, 2026
Merged

fix: require is_secret attribute on deploy environment variables#621
ndhoule merged 1 commit intomasterfrom
fix/deploy-variables-require-is-secret-attr

Conversation

@ndhoule
Copy link
Contributor

@ndhoule ndhoule commented Feb 18, 2026

This attribute is always required by the API and so omitting it, even
when false or defaulting to false, results in an API error.

I've also cleaned up an unused type from the porcelain package.

This attribute is always required by the API and so omitting it, even
when false or defaulting to false, results in an API error.

I've also cleaned up an unused type from the porcelain package.
@netlify
Copy link

netlify bot commented Feb 18, 2026

Deploy Preview for open-api ready!

Name Link
🔨 Latest commit 8a97666
🔍 Latest deploy log https://app.netlify.com/projects/open-api/deploys/69950ff81cd415000898bdf3
😎 Deploy Preview https://deploy-preview-621--open-api.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ndhoule ndhoule marked this pull request as ready for review February 18, 2026 01:07
@ndhoule ndhoule requested review from a team as code owners February 18, 2026 01:07
Copy link

@jaredm563 jaredm563 left a comment

Choose a reason for hiding this comment

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

Pain

@ndhoule ndhoule merged commit 964c698 into master Feb 18, 2026
14 checks passed
@ndhoule ndhoule deleted the fix/deploy-variables-require-is-secret-attr branch February 18, 2026 01:17
ndhoule pushed a commit that referenced this pull request Feb 18, 2026
🤖 I have created a release *beep* *boop*
---


##
[2.49.1](v2.49.0...v2.49.1)
(2026-02-18)


### Bug Fixes

* require is_secret attribute on deploy environment variables
([#621](#621))
([964c698](964c698))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: token-generator-app[bot] <82042599+token-generator-app[bot]@users.noreply.github.com>
Co-authored-by: token-generator-app[bot] <token-generator-app[bot]@users.noreply.github.com>
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

Comments