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

Collection of PR #2608

Closed
wants to merge 280 commits into from
Closed

Collection of PR #2608

wants to merge 280 commits into from

Conversation

burningtnt
Copy link
Member

@burningtnt burningtnt commented Dec 22, 2023

A collection of PRs to tell every collaborator that how much modifications is waiting for approval.
DO NOT merge or close this PR.

Download: https://zkitefly.github.io/HMCL-Snapshot-Update/prs

zkitefly and others added 30 commits July 1, 2023 01:41
# Conflicts:
#	HMCL/src/main/java/org/jackhuang/hmcl/game/HMCLGameRepository.java
# Conflicts:
#	HMCLCore/build.gradle.kts
# Conflicts:
#	HMCL/src/main/java/org/jackhuang/hmcl/Metadata.java
# Conflicts:
#	HMCL/src/main/resources/assets/lang/I18N_zh_CN.properties
# Conflicts:
#	HMCL/src/main/resources/assets/lang/I18N.properties
#	HMCL/src/main/resources/assets/lang/I18N_es.properties
#	HMCL/src/main/resources/assets/lang/I18N_zh.properties
#	HMCL/src/main/resources/assets/lang/I18N_zh_CN.properties
@Glavo
Copy link
Member

Glavo commented Dec 24, 2023

不觉得这个 PR 对审查有任何实质性帮助,而且它会一直对 watch 了 PR 动态的人不停发送 notification 和邮件,很烦。

@Glavo Glavo closed this Dec 24, 2023
@burningtnt
Copy link
Member Author

  1. 既然你选择关注 PR 动态,那你就要做好邮箱被打爆的心理预期
  2. 这个 PR 是用于告诉所有 Collaborator HMCL 目前有多少功能已经做出来了,但没有被合并进主线的
  3. 这个 PR 的构建版本也可以让急需某些功能的人先用上,因此会分发,才会通过 Hook 的形式更改应用名称和版本号

@Glavo

@Glavo
Copy link
Member

Glavo commented Dec 24, 2023

既然你选择关注 PR 动态,那你就要做好邮箱被打爆的心理预期

这个 PR 是用于告诉所有 Collaborator HMCL 目前有多少功能已经做出来了,但没有被合并进主线的

Watch PR 的动态是为了关注开发进展。本 PR 几百个提交和几百个文件更改堆在一起,到底提醒了啥告诉了啥我是看不出来,只看到了无意义的 notification 和邮件骚扰。

对应 PR 着急想合并可以在 PR 里艾特,可以在 discussions 或者 issue 里说,开个 PR 在这占坑没有任何正面意义,只是在无意义的消耗你和 watch 本项目 PR 的人的精力。

黄鱼忙着工作,yushi 近期写不了代码,我忙于以 JApp 为首的一些项目以及在 OpenJDK 那边推进一些变化,暂时也没空管这些,下个月我可能会多放一些精力在 HMCL 上。

这个 PR 的构建版本也可以让急需某些功能的人先用上,因此会分发,才会通过 Hook 的形式更改应用名称和版本号

要分发请维护在自己的仓库里,不需要开个 PR 在这占坑。

@burningtnt
Copy link
Member Author

image

280 条提交,近 5000 行更改,这难道没有反应出 HMCL 目前有很多功能已经做出来了,但没有被合并进主线

不过,就保持现在这个状态就 OK 了 ———— 不打开以避免占坑,不删除以避免无法注意到 commits 数量

@Glavo
Copy link
Member

Glavo commented Dec 24, 2023

280 条提交,近 5000 行更改,这难道没有反应出 HMCL 目前有很多功能已经做出来了,但没有被合并进主线

PR 列表就在这,开不开这个 PR 都没有影响。

审查和测试都是要消耗精力的,不合 PR 是因为需要时间充分的审查和测试。本 PR 并不会变出这些时间,对合并 PR 没有正面的意义。

@burningtnt
Copy link
Member Author

本分支的意义在于提供一个带有较多功能的预览版本,供部分人群使用。

对于意外造成的骚扰,我很抱歉。

@zkitefly zkitefly mentioned this pull request Dec 25, 2023
@zkitefly
Copy link
Member

zkitefly commented Dec 25, 2023

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.

3 participants