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

Docs: add issue-specification-document #59

Merged
merged 7 commits into from
Jul 6, 2023
Merged
Changes from 4 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 38 additions & 0 deletions docs/ISSUE_SPECIFICATION.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
# issue specification

- we use [lot issues] (https://github.com/openIMSDK/openKF/issues) before reporting bugs, Please make sure that has been
searching similar [question] (https://github.com/OpenIMSDK/OpenKF/issues), because they may have already answered or
Copy link
Contributor

@charles-chenzz charles-chenzz Jul 5, 2023

Choose a reason for hiding this comment

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

the link didn't embed
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maybe document is not compatible, i will check it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

already fix it

is being repaired. New questions should be directed to [Question Assistant](https://openIMSDK/issue-helper?
repo=openKF) submitted. For bug reports, include code that can be used to reproduce the problem and how to reproduce
it. For new feature suggestions, indicate the changes you want and the expected behavior.

## **for commiter**

Copy link
Contributor

Choose a reason for hiding this comment

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

typo, should be committer

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fix it already

##### **working for pr/mr**

Copy link
Contributor

Choose a reason for hiding this comment

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

pr only, this is not gitlab we don't have mr

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fix it already

- commiters should finish code unit test again and record test dirary and if nessary do somke test
- commiters should check for code
convenant [code convenant](https://github.com/OpenIMSDK/OpenKF/CONTRIBUTING.md)

##### **working for issue management**

- add labels to each issue and check it out.
- format issue style when needed.
- keep in touch with each contributor.
- Keep up with the status of the issue.
- assign assign one or more who to solve it.

## **for contributor**

##### **raise issue**

- developers should add full labels for issue you raised.(including **kind**(问题类型 such as,feat,bug),**Degree of
importance**(重要程度 emergency,important,normal),**strategy**(处置策略 such as self ,paticipant,coperation),*
*progress**(进度 such as running,later),**size**(问题规模 such as L,M,XXL))

##### **claim issue and meke pr**

Copy link
Contributor

Choose a reason for hiding this comment

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

typo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fix it already

- developers can claim it by comment (want to claim + issueId)
- when doing pr process, developer should make Incremental testing at first and then doing **/contributor.md 's**
process.

Loading