Skip to content

fix: add return source logic when file not downloaded and add loader …#1287

Merged
IWANABETHATGUY merged 1 commit into
rolldown:mainfrom
zhusjfaker:fix/script-gen-esbuild-test
May 30, 2024
Merged

fix: add return source logic when file not downloaded and add loader …#1287
IWANABETHATGUY merged 1 commit into
rolldown:mainfrom
zhusjfaker:fix/script-gen-esbuild-test

Conversation

@zhusjfaker

Copy link
Copy Markdown
Contributor

…suite

Description

in the old logic

when the script exec " return fs.readFileSync(sourcePath).toString()" and golang file not downloaded and happend exception

the logic will try to download file but not return the golang code source

i fix it and return the expect code

please review it thank you master and have nice day

@netlify

netlify Bot commented May 30, 2024

Copy link
Copy Markdown

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit f50a39b
🔍 Latest deploy log https://app.netlify.com/sites/rolldown-rs/deploys/66586dd720a38b0009619840

@zhusjfaker zhusjfaker force-pushed the fix/script-gen-esbuild-test branch 2 times, most recently from 27294dd to 075cbda Compare May 30, 2024 08:10
@zhusjfaker

Copy link
Copy Markdown
Contributor Author

sorry i dont have right to re_run workflow to check the change

@zhusjfaker

Copy link
Copy Markdown
Contributor Author

@IWANABETHATGUY please help me re_run this workflow ,i have no right to reboot it

@IWANABETHATGUY IWANABETHATGUY force-pushed the fix/script-gen-esbuild-test branch from 075cbda to f50a39b Compare May 30, 2024 12:15
@IWANABETHATGUY IWANABETHATGUY added this pull request to the merge queue May 30, 2024
Merged via the queue into rolldown:main with commit 103b8e5 May 30, 2024
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