Skip to content

Conversation

xiehan
Copy link
Member

@xiehan xiehan commented Aug 11, 2025

Back in May, all the contents of the website/ directory which powers our documentation on developer.hashicorp.com were migrated to the centralized hashicorp/web-unified-docs repository. At the time, we left the docs here just in case something happened and we needed to revert the migration, but we added a comment to every single docs page redirecting users to the new location in #3863.

Now that a couple of months have passed, we can confirm the migration was completed successfully and there's no reason to revert it, so it's time to completely delete all the contents of the old website/ directory, which can be a potential source of confusion for users.

Kudos to Radek for allowing me to copy your homework in hashicorp/terraform#37417.

Open questions/concerns

  • We had two Vercel projects running in this repo, one for the dev docs and the other for our cdk.tf redirects. I think the top-level vercel.json file refers to the dev docs and the one in the cdk.tf/ directory powers the redirects, but if anyone thinks the top-level file isn't safe to delete, let me know.
  • I think I figured out a "minimum viable implementation" to keep the generate-docs:api script working; it'll still generate the docs in an ignored folder, and then you'll have to manually copy-paste the contents in a PR on the web-unified-docs repo. I don't think it's possible to do a similar "minimum viable implementation" of the generate-docs:translations script, as that one relies on the old docs being present in the repo. While we might be able to do something with git submodules or symlinks, I don't have the bandwidth to figure that out, so I think it makes the most sense to just delete this tooling; anyone opposed to doing that?

@xiehan xiehan requested review from a team as code owners August 11, 2025 14:39
@xiehan xiehan requested review from mutahhir, ansgarm and EnMod and removed request for a team August 11, 2025 14:39
Copy link
Member

@ansgarm ansgarm left a comment

Choose a reason for hiding this comment

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

anyone opposed to doing that?

Nope, seems fine for me 👍

Copy link
Member

Choose a reason for hiding this comment

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

This one's safe to delete 👍 The one for the redirects is in cdk.tf/ as you assumed.

@EnMod
Copy link

EnMod commented Aug 12, 2025

Vercel builds have succeeded so this seems good to me, but I'd like to defer to those who have worked on hashicorp/web-unified-docs to see if they have anything to add.

@EnMod EnMod requested review from a team and LeahMarieBush and removed request for a team August 12, 2025 22:58
Copy link
Contributor

@LeahMarieBush LeahMarieBush left a comment

Choose a reason for hiding this comment

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

👍

@xiehan xiehan merged commit a36c83b into main Aug 13, 2025
917 of 925 checks passed
@xiehan xiehan deleted the remove-website-docs branch August 13, 2025 16:32
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