Skip to content

Fix devel planmodifier#1656

Merged
jaylonmcshan19-x merged 2 commits intohashicorp:mainfrom
jaylonmcshan19-x:fix-devel-planmodifier
Jun 23, 2025
Merged

Fix devel planmodifier#1656
jaylonmcshan19-x merged 2 commits intohashicorp:mainfrom
jaylonmcshan19-x:fix-devel-planmodifier

Conversation

@jaylonmcshan19-x
Copy link
Contributor

Rollback Plan

If a change needs to be reverted, we will publish an updated version of the library.

Changes to Security Controls

Are there any changes to security controls (access controls, encryption, logging) in this pull request? If so, explain.

Description

Previously, the suppressDevelPlanModifier would always return the state value
for devel when a version was set, regardless of whether the user explicitly
set devel = false. This change ensures that we only suppress devel if it was not explicitly set
in the Terraform configuration

Acceptance tests

  • Have you added an acceptance test for the functionality being added?

Release Note

Release note for CHANGELOG:

...

References

#1654

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

@jaylonmcshan19-x jaylonmcshan19-x requested a review from a team as a code owner June 23, 2025 15:41
@jaylonmcshan19-x jaylonmcshan19-x force-pushed the fix-devel-planmodifier branch from 12ffac0 to 42428fc Compare June 23, 2025 15:43
@jaylonmcshan19-x jaylonmcshan19-x merged commit 7f406c7 into hashicorp:main Jun 23, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants