Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: update solutions description #4948

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

chenjiahan
Copy link
Member

@chenjiahan chenjiahan commented Nov 14, 2023

Summary

Update solutions description, remove Modern.js Builder.

🤖 Generated by Copilot at 773ca97

This pull request removes any mention of Modern.js Builder, a deprecated solution, from the documentation files and the documentation site. It also introduces Remix as an alternative solution for web development and updates the recommendations for build tools and documentation sites. It affects the README.md, README.zh-CN.md, introduction.mdx, rspress.config.ts, index.module.scss, index.tsx files in different directories.

Details

🤖 Generated by Copilot at 773ca97

  • Remove Modern.js Builder from the system and update the introduction sections in the README and documentation files (link, link, link, link, link, link, link, link, link, link, link, link, link)
  • Add a section about Remix and a link to a discussion comparing Modern.js and Remix in the documentation files (link, link)
  • Change the navbar items for the documentation site to remove Modern.js Builder and add Rsbuild and Rspress in the rspress.config.ts file (link)
  • Add a style class for two cards in the index.module.scss file and apply it to the solution cards component in the index.tsx file when it only has two cards (link, link)
  • Remove the cards for Modern.js Builder and Rspress from the home page of the documentation site in the index.tsx file (link)

Related Issue

Checklist

  • I have added changeset via pnpm run change.
  • I have updated the documentation.
  • I have added tests to cover my changes.

Copy link

changeset-bot bot commented Nov 14, 2023

⚠️ No Changeset found

Latest commit: 773ca97

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

@chenjiahan chenjiahan requested a review from 9aoy November 14, 2023 09:12
@chenjiahan chenjiahan merged commit b10f384 into web-infra-dev:main Nov 14, 2023
8 checks passed
@chenjiahan chenjiahan deleted the solutions_desc_1114 branch November 14, 2023 11:15
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.

2 participants