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.

Sign up for GitHub

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

Jump to bottom

fix: fix some issues in module-federation/bridge #2791

Closed
wants to merge 13 commits into from
Closed

fix: fix some issues in module-federation/bridge #2791

wants to merge 13 commits into from

Conversation

danpeen
Copy link
Contributor

@danpeen danpeen commented Jul 24, 2024

Description

Related Issue

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have updated the documentation.

RussellCanfield and others added 13 commits July 12, 2024 10:19
@RussellCanfield
feat: add squadron-ai (#2708)
624ad4b
@ScriptedAlchemy
chore: align react version in runtime demo (#2763)
52433fe 
Co-authored-by: ScriptedAlchemy <zackaryjackson@bytedance.com>
@2heal1
fix(dts-plugin): support compile vue@3 dts (#2723)
b00ef13
@ScriptedAlchemy
fix: disable chunk split on next server (#2765)
6e85e36 
Co-authored-by: ScriptedAlchemy <zackaryjackson@bytedance.com>
@zhoushaw @github-actions @2heal1
Release v0.2.7 (#2770)
9d46f11 
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: 2heal1 <TwoHeal@163.com>
@2heal1
fix(dts-plugin): ensure dts file path if it has deep path (#2772)
f3d13c2
@zhoushaw @github-actions @2heal1
Release v0.2.8 (#2774)
4da7d33 
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: 2heal1 <TwoHeal@163.com>
@2heal1 @zhoushaw
feat: support modernjs ssr (#2482)
fa37cc4 
Co-authored-by: Zhou xiao <codingzx@gmail.com>
@2heal1
fix(dts-plugin): temp tsconfig path should extends original tsconfig (#…
7c4d99b 
…2780)
@zhoushaw @github-actions
Release v0.3.0 (#2782)
38d378a 
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@2heal1
chore: fix modernjs plugin version (#2783)
6ab4819
@2heal1
chore: remove useless log (#2784)
0a9adaf
@zhoushaw @github-actions @2heal1
Release v0.3.1 (#2785)
bc245f2 
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: 2heal1 <TwoHeal@163.com>
github-advanced-security[bot]
github-advanced-security bot found potential problems Jul 24, 2024
View reviewed changes
packages/dts-plugin/src/core/lib/typeScriptCompiler.ts
const execPromise = util.promisify(exec);
const cmd = `npx ${remoteOptions.compilerInstance} --project ${tempTsConfigJsonPath}`;
try {
await execPromise(cmd);

Check warning

Code scanning / CodeQL

Shell command built from environment values Medium

This shell command depends on an uncontrolled
absolute path
Loading
.
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Jul 24, 2024

⚠️ No Changeset found

Latest commit: bc245f2

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

@netlify Netlify
Copy link

netlify bot commented Jul 24, 2024 •
edited
Loading

✅ Deploy Preview for module-federation-docs ready!

Name Link
🔨 Latest commit bc245f2
🔍 Latest deploy log https://app.netlify.com/sites/module-federation-docs/deploys/66a0b5ab453c8600081c32e3
😎 Deploy Preview https://deploy-preview-2791--module-federation-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@danpeen danpeen changed the title fix: fix some issues in module-federation fix: fix some issues in module-federation/bridge Jul 24, 2024
@danpeen danpeen closed this Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
change: fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@danpeen @RussellCanfield @ScriptedAlchemy @2heal1 @zhoushaw