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

代码分离,功能优化 #204

Merged
merged 6 commits into from
Sep 23, 2024
Merged

代码分离,功能优化 #204

merged 6 commits into from
Sep 23, 2024

Conversation

Zhe28
Copy link
Contributor

@Zhe28 Zhe28 commented Sep 20, 2024

  1. 代码迁移到 ts.
  2. 分离 axios 代码.
  3. 增加 pinia 支持,全局状态代码迁移到相对应的 store.
  4. 代码格式优化, 用 === 代替 ==.
  5. 代码声明更改,用 const 代替 var 声明.
  6. Header 使用 Router 导航.
  7. v-for 增加 key.
  8. 用 prettier 格式化下代码.

我想问下不考虑迁移到 ts 下吗?
还有就是代码格式化的问题,弄个约束代码风格的东西,有时候真是挑拣着提交的, 很不方便。

1. 代码迁移到 ts.
2. 分离 axios 部分代码.
3. 增加 pinia 支持,全局状态代码迁移到相对应的 store.
4. 代码格式优化, 用 === 代替 ==.
5. 代码声明更改,用 const 代替 var 声明.
6. Header 使用 Router 导航.
7. v-for 增加 key.
@Jinnrry
Copy link
Owner

Jinnrry commented Sep 21, 2024

没用ts是因为我没写过ts,我不太熟悉ts。如果你愿意帮忙改的话可以迁移。

代码格式你可以加一个

不过自动化测试编译好像没过,你看看测试报错呢

@Zhe28
Copy link
Contributor Author

Zhe28 commented Sep 21, 2024

没用ts是因为我没写过ts,我不太熟悉ts。如果你愿意帮忙改的话可以迁移。

代码格式你可以加一个

不过自动化测试编译好像没过,你看看测试报错呢

我看里面是之前保留的 :prop="defaultProp" 属性,我没找到相关的引用,我就把他给删了。
这个是留什么东西的吗?

@Jinnrry Jinnrry merged commit ad0167f into Jinnrry:master Sep 23, 2024
1 check passed
@Jinnrry
Copy link
Owner

Jinnrry commented Sep 23, 2024

2个PR都合了,看改动量比较大,发了个2.7.0的版本号。

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