Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

refactor: allow using server statically #1457

Merged
merged 1 commit into from
Jan 28, 2022

Conversation

pan93412
Copy link
Contributor

將原本 "NeteaseCloudMusicApi/server" 掃描 modules 目錄的部分抽出 server.js,
改移到 main.js。這樣使用者就有辦法只載入靜態的伺服器部分(不動態載入模組)。

這個 patch 的 breaking changes 不易發生。可作為 patch version 發佈。

@vercel
Copy link

vercel bot commented Jan 28, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

netease-cloud-music-api-binaryify – ./

🔍 Inspect: https://vercel.com/binaryify/netease-cloud-music-api-binaryify/BRb5TymBtEqfdFJXw1Q4NyQPcJjn
✅ Preview: https://netease-cloud-music-api-binaryify-git-fork-pan-0c8b48-binaryify.vercel.app

netease-cloud-music-api – ./docs

🔍 Inspect: https://vercel.com/binaryify/netease-cloud-music-api/97Akwp2yyA2g28DmFbvLJ1VRjRvr
✅ Preview: https://netease-cloud-music-api-git-fork-pan93412-refa-5f6dc5-binaryify.vercel.app

@pan93412
Copy link
Contributor Author

r? @Binaryify 謝謝 🙏

@pan93412 pan93412 marked this pull request as draft January 28, 2022 04:49
將原本 "NeteaseCloudMusicApi/server"
掃描 modules 目錄的部分抽出 server.js,
改移到 main.js。這樣使用者就有辦法只載入
靜態的伺服器部分(不動態載入模組)。

這個 patch 的 breaking changes 不易發生。
可作為 patch version 發佈。
@Binaryify
Copy link
Owner

嗯,这样是会更好点

@Binaryify Binaryify merged commit d665429 into Binaryify:master Jan 28, 2022
@pan93412 pan93412 deleted the refactor/dynamic-to-main branch January 28, 2022 05:54
Binaryify added a commit that referenced this pull request Jan 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants