Skip to content

Conversation

@Young-Lord
Copy link
Contributor

@github-actions
Copy link

Hi @Young-Lord,

❤️ 感谢你的贡献!你的 PR 当前基于 main 分支,请修改使用 dev 分支

@Young-Lord Young-Lord changed the base branch from main to dev August 26, 2025 06:45
Copy link
Contributor

@ccmywish ccmywish left a comment

Choose a reason for hiding this comment

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

👍👍👍 感谢提交上海交通大学镜像站的源以及对该 recipe 的关注。

还得在 chsrc-main.c 中 register 一下你的信息,@Young-Lord 这个 ID 才会在运行时生效。四个参数含义可见 struct Contributor_t

https://github.com/RubyMetric/chsrc/blob/main/src/chsrc-main.c#L78

@Young-Lord
Copy link
Contributor Author

还得在 chsrc-main.c 中 register 一下你的信息,@Young-Lord 这个 ID 才会在运行时生效

好的,我会在几个小时内同步上游更改后force push更改。
另外,是否应当将“修改 chsrc-main.c”这一步加到贡献指南里?

@ccmywish
Copy link
Contributor

另外,是否应当将“修改 chsrc-main.c”这一步加到贡献指南里?

可以的,作为第一次提交贡献的朋友,如果你在初次参与过程中感受到整个流程有哪些不清楚的或者困难的,都可以提问以及记录在文档里。🤝

@Young-Lord
Copy link
Contributor Author

可以了,文档的话我会稍后另开一个pr。

@ccmywish ccmywish added this to the v0.2.3 milestone Aug 27, 2025
@ccmywish ccmywish added os_target os target labels Aug 27, 2025
@ccmywish ccmywish merged commit 95de30d into RubyMetric:dev Aug 27, 2025
@ccmywish
Copy link
Contributor

Happy changing source~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

os_target os target

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants