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

refactor: 重构主题色配置功能 #110

Merged
merged 3 commits into from
Sep 29, 2024
Merged

Conversation

xixiIBN5100
Copy link
Collaborator

No description provided.

@xixiIBN5100 xixiIBN5100 merged commit 1c2a44d into zjutjh:master Sep 29, 2024
1 check failed
Copy link
Collaborator

Choose a reason for hiding this comment

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

page.config.js 不是放在 src 目录下的呀。它是放在每个页面的文件夹下,应该是用来替换 index.config.ts

@j10ccc
Copy link
Collaborator

j10ccc commented Sep 29, 2024

feat:更新Taro版本,新增page.config.ts文件 这个 commit 是不是没有必要交上来啊?

xixiIBN5100 added a commit that referenced this pull request Sep 29, 2024
@xixiIBN5100
Copy link
Collaborator Author

feat:更新Taro版本,新增page.config.ts文件 这个 commit 是不是没有必要交上来啊?

这个commit改了版本会导致action爆红,我在本地用第二个commit覆盖了修改然后合并的时候squash了一下.今天本地和远程仓库的commit出现了分歧进行了reset操作用的基准是这个commit,确实应该先把这个回退一下,之前不知到有检测缓存的action在本地更新了依赖以后也直接给组织的仓库pr了

<text
v-for="item in hadThemeList"
Copy link
Collaborator

Choose a reason for hiding this comment

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

这里有类型错误,hadThemeList 的类型不是 ArrayLike @xixiIBN5100

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