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

允许部署在子目录下 #171

Merged
merged 1 commit into from
Jan 12, 2025
Merged

允许部署在子目录下 #171

merged 1 commit into from
Jan 12, 2025

Conversation

IamIpanda
Copy link
Contributor

@IamIpanda IamIpanda commented Jan 8, 2025

此修改允许app部署在子目录形如 https://a.b.com/sql/ 下。

参见:https://cn.vitejs.dev/guide/build.html#relative-base

Summary by Sourcery

部署:

  • 容許將應用程式部署喺子目錄,例如 https://a.b.com/sql/
Original summary in English

Summary by Sourcery

Deployment:

  • Allow deploying the application under a subdirectory, such as https://a.b.com/sql/ .

Copy link

sonarqubecloud bot commented Jan 8, 2025

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

@IamIpanda - 我睇咗你嘅改動,睇落好正!

以下係我喺審查時睇嘅嘢
  • 🟢 一般問題:一切睇落都冇問題
  • 🟢 安全性:一切睇落都冇問題
  • 🟢 測試:一切睇落都冇問題
  • 🟢 複雜度:一切睇落都冇問題
  • 🟢 文檔:一切睇落都冇問題

Sourcery 對開源項目係免費嘅 - 如果你鍾意我哋嘅審查,請考慮分享佢哋 ✨
幫我變得更有用!請喺每個留言度撳 👍 或 👎,我會用呢個回饋嚟改善你嘅審查。
Original comment in English

Hey @IamIpanda - I've reviewed your changes and they look great!

Here's what I looked at during the review
  • 🟢 General issues: all looks good
  • 🟢 Security: all looks good
  • 🟢 Testing: all looks good
  • 🟢 Complexity: all looks good
  • 🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@Stapxs Stapxs changed the base branch from next to fun/202412 January 12, 2025 08:21
@Stapxs Stapxs merged commit c1b3b3b into Stapxs:fun/202412 Jan 12, 2025
3 checks passed
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