Skip to content
This repository has been archived by the owner on Oct 25, 2024. It is now read-only.

update contricution.md #121

Merged
merged 6 commits into from
Aug 11, 2023
Merged

update contricution.md #121

merged 6 commits into from
Aug 11, 2023

Conversation

VincyZhang
Copy link
Contributor

Type of Change

documentation

Description

update contribution rules

Expected Behavior & Potential Risk

no

How has this PR been tested?

spell check

Dependency Change?

no

@VincyZhang VincyZhang force-pushed the contribution_document branch 2 times, most recently from 689b83a to 0f3ddf9 Compare August 10, 2023 16:56
Signed-off-by: Wenxin Zhang <wenxin.zhang@intel.com>
@VincyZhang VincyZhang force-pushed the contribution_document branch from 0f3ddf9 to 48061f7 Compare August 10, 2023 16:57
Signed-off-by: Wenxin Zhang <wenxin.zhang@intel.com>
Signed-off-by: Wenxin Zhang <wenxin.zhang@intel.com>
@VincyZhang VincyZhang force-pushed the contribution_document branch 6 times, most recently from 36970c6 to f34b97f Compare August 11, 2023 03:27
Signed-off-by: Wenxin Zhang <wenxin.zhang@intel.com>
@VincyZhang VincyZhang force-pushed the contribution_document branch from f34b97f to 514a082 Compare August 11, 2023 03:34
Copy link
Contributor

@kevinintel kevinintel left a comment

Choose a reason for hiding this comment

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

some comments

docs/contributions.md Outdated Show resolved Hide resolved
docs/contributions.md Outdated Show resolved Hide resolved
docs/CI_introduction.md Show resolved Hide resolved
docs/CI_introduction.md Show resolved Hide resolved
Copy link
Contributor

@kevinintel kevinintel left a comment

Choose a reason for hiding this comment

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

LGTM

@hshen14 hshen14 merged commit d6497ed into main Aug 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants