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

feat: add @vuepress/bundler-farm #1583

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

typed-sigterm
Copy link
Contributor

@typed-sigterm typed-sigterm commented Jul 1, 2024

🚧 Experimental + Work in Progress 🚧

Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Provide a description in this PR that addresses what the PR is solving. If this PR is going to solve an existing issue, please reference the issue (e.g. close #123).

What is the purpose of this pull request?

  • Bug fix
  • New feature
  • Documentation update
  • Other

Description

Ref: #1552

@coveralls
Copy link

coveralls commented Jul 1, 2024

Pull Request Test Coverage Report for Build 9739675060

Details

  • 0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.03%) to 38.576%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/vite-kit/src/plugins/vuepressVuePlugin.ts 0 2 0.0%
Totals Coverage Status
Change from base Build 9656194136: -0.03%
Covered Lines: 631
Relevant Lines: 1663

💛 - Coveralls

@coveralls
Copy link

coveralls commented Jul 1, 2024

Pull Request Test Coverage Report for Build 9739790629

Details

  • 0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.03%) to 38.576%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/vite-kit/src/plugins/vuepressVuePlugin.ts 0 2 0.0%
Totals Coverage Status
Change from base Build 9656194136: -0.03%
Covered Lines: 631
Relevant Lines: 1663

💛 - Coveralls

@Mister-Hope
Copy link
Member

Looking forward to it (๑•ૅω•´๑)

@coveralls
Copy link

coveralls commented Jul 1, 2024

Pull Request Test Coverage Report for Build 9740770751

Details

  • 0 of 28 (0.0%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.6%) to 38.04%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/bundler-farm/src/resolveFarmConfig.ts 0 2 0.0%
packages/vite-kit/src/plugins/vuepressVuePlugin.ts 0 2 0.0%
packages/bundler-farm/src/farmBundler.ts 0 3 0.0%
packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts 0 3 0.0%
packages/bundler-farm/src/dev.ts 0 7 0.0%
packages/bundler-farm/src/build.ts 0 11 0.0%
Totals Coverage Status
Change from base Build 9656194136: -0.6%
Covered Lines: 631
Relevant Lines: 1689

💛 - Coveralls

@coveralls
Copy link

coveralls commented Jul 1, 2024

Pull Request Test Coverage Report for Build 9741133313

Details

  • 0 of 29 (0.0%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.6%) to 38.025%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/bundler-farm/src/resolveFarmConfig.ts 0 2 0.0%
packages/vite-kit/src/plugins/vuepressVuePlugin.ts 0 2 0.0%
packages/bundler-farm/src/farmBundler.ts 0 3 0.0%
packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts 0 3 0.0%
packages/bundler-farm/src/dev.ts 0 8 0.0%
packages/bundler-farm/src/build.ts 0 11 0.0%
Totals Coverage Status
Change from base Build 9656194136: -0.6%
Covered Lines: 631
Relevant Lines: 1690

💛 - Coveralls

@coveralls
Copy link

coveralls commented Jul 2, 2024

Pull Request Test Coverage Report for Build 9756258638

Details

  • 0 of 29 (0.0%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.6%) to 38.025%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/bundler-farm/src/resolveFarmConfig.ts 0 2 0.0%
packages/vite-kit/src/plugins/vuepressVuePlugin.ts 0 2 0.0%
packages/bundler-farm/src/farmBundler.ts 0 3 0.0%
packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts 0 3 0.0%
packages/bundler-farm/src/dev.ts 0 8 0.0%
packages/bundler-farm/src/build.ts 0 11 0.0%
Totals Coverage Status
Change from base Build 9656194136: -0.6%
Covered Lines: 631
Relevant Lines: 1690

💛 - Coveralls

@coveralls
Copy link

coveralls commented Jul 2, 2024

Pull Request Test Coverage Report for Build 9756442612

Details

  • 0 of 30 (0.0%) changed or added relevant lines in 7 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.6%) to 37.965%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/vite-kit/src/plugins/vuepressMainPlugin.ts 0 1 0.0%
packages/bundler-farm/src/resolveFarmConfig.ts 0 2 0.0%
packages/vite-kit/src/plugins/vuepressVuePlugin.ts 0 2 0.0%
packages/bundler-farm/src/farmBundler.ts 0 3 0.0%
packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts 0 3 0.0%
packages/bundler-farm/src/dev.ts 0 8 0.0%
packages/bundler-farm/src/build.ts 0 11 0.0%
Totals Coverage Status
Change from base Build 9656194136: -0.6%
Covered Lines: 631
Relevant Lines: 1690

💛 - Coveralls

Copy link
Member

@meteorlxy meteorlxy left a comment

Choose a reason for hiding this comment

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

Really interesting. Didn't know farm bundler before. Looking forward to the integration. 👍

.github/workflows/e2e.yml Outdated Show resolved Hide resolved
@@ -39,6 +39,7 @@
"@vitest/coverage-istanbul": "^1.6.0",
"bumpp": "^9.4.1",
"conventional-changelog-cli": "^5.0.0",
"core-js": "^3.37.1",
Copy link
Member

Choose a reason for hiding this comment

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

May not put it in root?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe upstream bug/design 🤔 Farm currently starts resolving core-js from each dependency's node_modules, which is a problem when using Monorepo. So put core-js in the root package to ensure core-js can be resolved and not unnecessarily increase the number of dependencies on other packages.

@typed-sigterm typed-sigterm force-pushed the feat/farm-bundler branch 2 times, most recently from 5c5b0bd to c9af9f6 Compare July 5, 2024 01:35
@coveralls
Copy link

coveralls commented Jul 5, 2024

Pull Request Test Coverage Report for Build 9801782315

Details

  • 0 of 30 (0.0%) changed or added relevant lines in 7 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.7%) to 37.935%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/vite-kit/src/plugins/vuepressMainPlugin.ts 0 1 0.0%
packages/bundler-farm/src/resolveFarmConfig.ts 0 2 0.0%
packages/vite-kit/src/plugins/vuepressVuePlugin.ts 0 2 0.0%
packages/bundler-farm/src/farmBundler.ts 0 3 0.0%
packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts 0 3 0.0%
packages/bundler-farm/src/dev.ts 0 8 0.0%
packages/bundler-farm/src/build.ts 0 11 0.0%
Totals Coverage Status
Change from base Build 9656194136: -0.7%
Covered Lines: 631
Relevant Lines: 1690

💛 - Coveralls

@coveralls
Copy link

coveralls commented Jul 5, 2024

Pull Request Test Coverage Report for Build 9801773016

Details

  • 0 of 30 (0.0%) changed or added relevant lines in 7 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.7%) to 37.935%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/vite-kit/src/plugins/vuepressMainPlugin.ts 0 1 0.0%
packages/bundler-farm/src/resolveFarmConfig.ts 0 2 0.0%
packages/vite-kit/src/plugins/vuepressVuePlugin.ts 0 2 0.0%
packages/bundler-farm/src/farmBundler.ts 0 3 0.0%
packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts 0 3 0.0%
packages/bundler-farm/src/dev.ts 0 8 0.0%
packages/bundler-farm/src/build.ts 0 11 0.0%
Totals Coverage Status
Change from base Build 9656194136: -0.7%
Covered Lines: 631
Relevant Lines: 1690

💛 - Coveralls

@coveralls
Copy link

coveralls commented Jul 5, 2024

Pull Request Test Coverage Report for Build 9801846027

Details

  • 0 of 30 (0.0%) changed or added relevant lines in 7 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.7%) to 37.935%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/vite-kit/src/plugins/vuepressMainPlugin.ts 0 1 0.0%
packages/bundler-farm/src/resolveFarmConfig.ts 0 2 0.0%
packages/vite-kit/src/plugins/vuepressVuePlugin.ts 0 2 0.0%
packages/bundler-farm/src/farmBundler.ts 0 3 0.0%
packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts 0 3 0.0%
packages/bundler-farm/src/dev.ts 0 8 0.0%
packages/bundler-farm/src/build.ts 0 11 0.0%
Totals Coverage Status
Change from base Build 9656194136: -0.7%
Covered Lines: 631
Relevant Lines: 1690

💛 - Coveralls

@typed-sigterm
Copy link
Contributor Author

This PR may take longer to complete. Vite SSR and Farm SSR are not very similar :(

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