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: Kotlinify core #1207

Merged
merged 4 commits into from
Jan 30, 2024
Merged

refactor: Kotlinify core #1207

merged 4 commits into from
Jan 30, 2024

Conversation

nopdan
Copy link
Contributor

@nopdan nopdan commented Jan 26, 2024

Pull request

Issue tracker

main changed files:

  • Trime.java
  • Speech.java

#1162 也有提交,将其拆分出来,优先合并

Feature

Describe features of pull request

Code of conduct

Style lint

  • make sytle-lint

Build pass

  • make debug

Manually test

  • Done

Code Review

  1. No wildcards import
  2. Manual build and test pass
  3. GitHub action ci pass
  4. At least one contributor reviews and votes
  5. Can be merged clean without conflicts
  6. PR will be merged by rebase upstream base

Daily build

Login and download artifact at https://github.com/osfans/trime/actions

Additional Info

@nopdan nopdan mentioned this pull request Jan 27, 2024
4 tasks
@goofyz
Copy link
Collaborator

goofyz commented Jan 27, 2024

this one need more time to test

@nopdan nopdan mentioned this pull request Jan 27, 2024
4 tasks
@nopdan
Copy link
Contributor Author

nopdan commented Jan 27, 2024

this one need more time to test

如果需要测试可以使用 #1211,该 pr 包含了另外两个提交

@nopdan nopdan marked this pull request as draft January 28, 2024 02:34
@nopdan nopdan marked this pull request as ready for review January 29, 2024 04:35
@Bambooin
Copy link
Collaborator

No issue found during daily usage, for kotlin code, beyond my scope.

Need someone with kotlin to help code review.

@Bambooin Bambooin merged commit 5833dba into osfans:develop Jan 30, 2024
3 checks passed
@nopdan nopdan deleted the kt-core branch February 3, 2024 09:30
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.

None yet

4 participants