Skip to content

Conversation

@dsa0x
Copy link
Member

@dsa0x dsa0x commented Sep 16, 2025

Fixes #

Target Release

1.14.x

Rollback Plan

  • If a change needs to be reverted, we will roll out an update to the code within 7 days.

Changes to Security Controls

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

CHANGELOG entry

  • This change is user-facing and I added a changelog entry.
  • This change is not user-facing.

@dsa0x dsa0x added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Sep 16, 2025
Base automatically changed from sams/list-no-depends-on to main September 16, 2025 11:40
@dsa0x dsa0x force-pushed the sams/list-remove-experiments branch 3 times, most recently from 5689b0f to 3589da0 Compare September 16, 2025 13:08
@dsa0x dsa0x marked this pull request as ready for review September 16, 2025 13:26
@dsa0x dsa0x requested review from a team as code owners September 16, 2025 13:26
@dbanck
Copy link
Member

dbanck commented Sep 16, 2025

I think we also need to move the query command out of experimental and enable the list schema output for terraform providers schema -json

@dbanck
Copy link
Member

dbanck commented Sep 16, 2025

What do you think about doing the query file parsing changes in this PR too? That way we don't have something on main that behaves in unintended ways

@dbanck dbanck added the 1.14-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged label Sep 16, 2025
@dbanck dbanck force-pushed the sams/list-remove-experiments branch 3 times, most recently from 458c6b0 to 53fb648 Compare September 16, 2025 19:23
@dbanck dbanck force-pushed the sams/list-remove-experiments branch from 53fb648 to 75697c8 Compare September 16, 2025 19:24
@dsa0x dsa0x merged commit 719aefd into main Sep 17, 2025
10 of 11 checks passed
@dsa0x dsa0x deleted the sams/list-remove-experiments branch September 17, 2025 10:29
dbanck pushed a commit that referenced this pull request Sep 17, 2025
dbanck added a commit that referenced this pull request Sep 17, 2025
@github-actions
Copy link
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

1.14-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged no-changelog-needed Add this to your PR if the change does not require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants