Skip to content

doc(command): follow rename of module.tofu - #112

Merged
diofeher merged 2 commits into
opentofu:mainfrom
billf:tofu-version-request-handler
Sep 3, 2025
Merged

doc(command): follow rename of module.tofu#112
diofeher merged 2 commits into
opentofu:mainfrom
billf:tofu-version-request-handler

Conversation

@billf

@billf billf commented Aug 27, 2025

Copy link
Copy Markdown
Contributor

In #80, module.tofu was renamed to module.opentofu, but the documentation was not updated to reflect this change.
As there exists a mechanism for redirecting deprecated commands, avail ourselves of it.

  • fix(command): docs mismatch for module.opentofu
  • chore(command): skip copy by adjusting types

Checklist

  • I have read the contribution guide.
  • I have not used an AI coding assistant to create this PR.
  • I have written all code in this PR myself OR I have marked all code I have not written myself (including modified code, e.g. copied from other places and then modified) with a comment indicating where it came from.
  • I (and other contributors to this PR) have not looked at the Terraform source code while implementing this PR.
  • If I'm releasing, I have read the releasing guide.

Go checklist

  • I have run golangci-lint on my change and receive no errors relevant to my code.
  • I have run existing tests to ensure my code doesn't break anything.
  • I have added tests for all relevant use cases of my code, and those tests are passing.
  • I have only exported functions, variables and structs that should be used from other packages.
  • I have added meaningful comments to all exported functions, variables, and structs.

billf added 2 commits August 27, 2025 12:33
Redirect handler removed in opentofu#80 c259357

Signed-off-by: bill fumerola <bill@fumerola.com>
Signed-off-by: bill fumerola <bill@fumerola.com>
@billf
billf force-pushed the tofu-version-request-handler branch from 53b59c6 to b1c9142 Compare August 27, 2025 19:33
@billf
billf marked this pull request as ready for review August 29, 2025 02:27
@billf
billf requested a review from a team as a code owner August 29, 2025 02:27

@Gogotchuri Gogotchuri left a comment

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.

Thanks for catching this and taking the time to submit this PR. We appreciate community involvement.

This is a valid thing that I have overlooked, and I am OK with merging this PR.


For future contributions, please refer to our Contributing Guide.
We generally do not accept PRs without first having an accepted issue for it. So, in the future, please open an issue first to discuss it before opening a PR.

@Gogotchuri

Copy link
Copy Markdown
Contributor

@diofeher If you also approve this, feel free to merge.

@diofeher diofeher left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for your contribution, @billf ! And good catch on refactoring the additionalHandlers code.

@diofeher
diofeher merged commit 8e68113 into opentofu:main Sep 3, 2025
21 checks passed
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.

4 participants