Skip to content

Conversation

@wickdynex
Copy link
Contributor

Description

Update PR default template content, add new section and examples also related issue to this template.

Additional Notes

If there's some mistake I made, plz comment, and I'll fix it and polish until is ready to be merged. Thanks😊.

@ccmywish ccmywish added this to the v0.2.0 milestone Dec 15, 2024
@ccmywish
Copy link
Contributor

Wow, such a detailed PR, thank you! 👍


  1. In your example 变更内容 and 描述 are too alike
  2. 相关问题 should be put into 描述
  3. 备注 and 风险与注意事项 can merge
  4. We must have a 方案 and 实现 section, this is very important
  5. 测试说明 should be the last section, simply renamed as 测试
  6. There is no need for a 总结

Please change it to this, and merge your content according to it.

### 描述

(1)问题的背景
(2)相关issue
(3)这个PR做了什么

### 方案

...

### 实现

...

### 注意

...

### 测试

@wickdynex
Copy link
Contributor Author

Hi @ccmywish ,
I've polished it and modified as your comment, how about now?😊

@ccmywish ccmywish changed the base branch from main to dev December 16, 2024 05:56
@ccmywish ccmywish merged commit c152e4d into RubyMetric:dev Dec 16, 2024
@ccmywish
Copy link
Contributor

Now new pull requests can have a solid base, thank you very much! 👍

@wickdynex
Copy link
Contributor Author

Now new pull requests can have a solid base, thank you very much! 👍

Welcome🤗

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