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(projects): add port env variables #607

Closed
wants to merge 3 commits into from

Conversation

loicduong
Copy link

No description provided.

@Azir-11
Copy link
Member

Azir-11 commented Sep 3, 2024

冲突的情况下这个prot会自动往后顺延的,没有很理解这个的意义

@NHZEX
Copy link
Contributor

NHZEX commented Sep 4, 2024

冲突的情况下这个prot会自动往后顺延的,没有很理解这个的意义

有意义,多个项目同时维护时自动解决 port 冲突是不可靠的,容易产生混乱,影响体验,而且 port 占用不一定是项目自身导致的,而是第三方程序。

@honghuangdc
Copy link
Member

目前想更改端口直接改 vite.config.ts ,当然放到环境文件也是可以的

不过目前 SoybeanAdmin 打算处于维护bug阶段,暂时不做功能上的变更,等后续大版本变更的再做修改

@honghuangdc
Copy link
Member

要更新上游代码,请用git rebase main去更新,不推荐用git merge,产生没必要的merge记录

@Azir-11
Copy link
Member

Azir-11 commented Sep 5, 2024

拉远程代码用git pull --rebase,可以把远程别人提交的代码压到自己提交的下面

@@ -44,5 +44,11 @@ VITE_STATIC_SUPER_ROLE=R_SUPER
# sourcemap
VITE_SOURCE_MAP=N

# Used to differentiate storage across different domains
# used to differentiate storage across different domains
Copy link
Member

Choose a reason for hiding this comment

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

这个变更丢掉

@honghuangdc
Copy link
Member

在最新代码上,重新提一个PR

@honghuangdc honghuangdc closed this Sep 7, 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.

4 participants