Skip to content

Conversation

@alcuadrado
Copy link
Member

This PR removes the website, as it's now in https://github.com/NomicFoundation/hardhat-website

@alcuadrado alcuadrado added the no changeset needed This PR doesn't require a changeset label Apr 17, 2025
@changeset-bot
Copy link

changeset-bot bot commented Apr 17, 2025

⚠️ No Changeset found

Latest commit: 8c7f53e

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@alcuadrado alcuadrado force-pushed the remove-website-from-main branch from 13908aa to 52fc2d2 Compare April 17, 2025 19:15
@alcuadrado alcuadrado force-pushed the remove-website-from-main branch from fc8d488 to 8c7f53e Compare April 17, 2025 19:44
@alcuadrado
Copy link
Member Author

@galargh can you take a look at the last two commits here? They are pretty small despite the PR being huge.

The first commit is just rm -rf docs/, but tons of changes from git's perspective.

@alcuadrado alcuadrado requested review from Copilot and galargh April 17, 2025 19:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes all website-related configurations and documentation files from the repository as the website has been migrated to a new location.

  • Removed website configuration files such as redirects.config.js, next.config.js, next-sitemap.js, and README.md
  • Removed Storybook, ESLint configurations, and website-specific installation steps from the workflow

Reviewed Changes

Copilot reviewed 426 out of 437 changed files in this pull request and generated no comments.

Show a summary per file
File Description
docs/redirects.config.js Removed the website redirects configuration file
docs/next.config.js Removed Next.js configuration file that referenced the website
docs/next-sitemap.js Removed sitemap configuration
docs/README.md Removed website documentation
docs/.storybook/preview.js Removed Storybook preview configuration
docs/.storybook/main.js Removed Storybook main configuration
docs/.storybook/common-decorators.js Removed Storybook decorator configuration
docs/.eslintrc.js Removed ESLint configuration for the website
.github/workflows/lint.yml Removed website-related linting steps from the workflow
Files not reviewed (11)
  • .github/workflows/check-docs-site.yml: Language not supported
  • docs/.babelrc: Language not supported
  • docs/.env.example: Language not supported
  • docs/.gitignore: Language not supported
  • docs/.npmrc: Language not supported
  • docs/.prettierignore: Language not supported
  • docs/package.json: Language not supported
  • docs/pnpm-workspace.yaml: Language not supported
  • docs/public/robots.txt: Language not supported
  • docs/public/sitemap.xml: Language not supported
  • docs/public/static/privacy-policy.html: Language not supported

@schaable schaable merged commit 52f091e into main Apr 18, 2025
8 checks passed
@schaable schaable deleted the remove-website-from-main branch April 18, 2025 15:54
@github-project-automation github-project-automation bot moved this from Backlog to Done in Hardhat Apr 18, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no changeset needed This PR doesn't require a changeset

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants