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

中英文文档合并以及Hyperledger文件重命名 #147

Merged
merged 2 commits into from
Mar 27, 2021

Conversation

davidkhala
Copy link
Member

@davidkhala davidkhala commented Feb 3, 2021

格式更新
大型改动

@SamYuan1990
Copy link
Member

@davidkhala , I hope in this pr we can have some document upgrade as mentioned in #116

@davidkhala
Copy link
Member Author

@SamYuan1990 but #116 has ambitious beyond tape as probe is also included.

@SamYuan1990
Copy link
Member

@SamYuan1990 but #116 has ambitious beyond tape as probe is also included.

I see, or could you please help with #131 ? To be honest... I hope we can have a pr with large scope upgrade for document.

@guoger
Copy link
Member

guoger commented Feb 5, 2021

@davidkhala commit需要sign-off

@davidkhala davidkhala marked this pull request as draft February 5, 2021 06:12
@davidkhala davidkhala force-pushed the davidkhala-patch-1 branch 2 times, most recently from c9dcc1e to 6bae863 Compare February 5, 2021 10:07
@davidkhala davidkhala changed the title Update whatis.md 中英文文档合并以及Hyperledger文件重命名 Feb 5, 2021
@davidkhala davidkhala marked this pull request as ready for review February 5, 2021 10:15
Comment on lines -124 to -147
## Help us improve

If you wish for new features or encounter any bug, please feel free to open [issue](https://github.com/guoger/tape/issues), and we always welcome [pull request](https://github.com/guoger/tape/pulls).
## [How to Contribute](CONTRIBUTING.md)

If you are reporting an issue, please generously turn on debug log with `export TAPE_LOGLEVEL=debug` and paste log in the issue
## [Maintainers](MAINTAINERS.md)

## Development
<details>
<summary>Click to expand</summary>

### Tape workflow
## LICENSE
Hyperledger Project source code files are made available under the Apache License, Version 2.0 (Apache-2.0), located in the [LICENSE](LICENSE) file.

Tape consists of several workers that run in goroutines, so that the pipeline is highly concurrent and scalable. Workers are connected via buffered channels, so they can pass products around.

![tape workflow](./docs/images/tape.jpeg)
</details>

## Maintainers

| Name | mail | github-ID
| ------ | ------------------------ | ----------- |
| Jay Gou | guojiannan1101@gmail.com | guoger |
| Sam Yuan | yy19902439@126.com | SamYuan1990 |
| Stone Cheng | chengyang418@163.com | stone-ch |
Copy link
Member

Choose a reason for hiding this comment

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

not sure why we need to delete this section? IMO this makes readme more complete

Copy link
Member Author

Choose a reason for hiding this comment

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

I planned to move this section to MAINTAINERS.md.
How about we set the title ##Maintainers here as a link to MAINTAINERS.md ?

@@ -0,0 +1,7 @@
# Maintainers 维护者信息

Copy link
Member Author

Choose a reason for hiding this comment

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

@guoger 郭哥,维护者移到这里了

@davidkhala davidkhala marked this pull request as ready for review February 14, 2021 04:31
@davidkhala davidkhala force-pushed the davidkhala-patch-1 branch 2 times, most recently from a28f81a to ea62ad7 Compare February 16, 2021 01:14
stone-ch
stone-ch previously approved these changes Mar 16, 2021
Copy link
Member

@stone-ch stone-ch left a comment

Choose a reason for hiding this comment

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

LGTM
Make the README clean. And actually it is necessary to add the complete English documents, too.

stone-ch
stone-ch previously approved these changes Mar 16, 2021
@stone-ch stone-ch enabled auto-merge (squash) March 16, 2021 14:11
格式更新
大型改动

Signed-off-by: David Liu <david.yx.liu@oracle.com>
@davidkhala
Copy link
Member Author

@stone-ch 刚才git reset的时候选错了方式,把代码搞乱了,现在应该好了

@stone-ch stone-ch requested review from guoger and stone-ch and removed request for guoger and SamYuan1990 March 16, 2021 15:14
@davidkhala davidkhala requested a review from guoger March 16, 2021 15:18
@stone-ch stone-ch merged commit afeafb4 into master Mar 27, 2021
@stone-ch stone-ch deleted the davidkhala-patch-1 branch March 27, 2021 08:56
@davidkhala
Copy link
Member Author

close #116

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

Successfully merging this pull request may close these issues.

4 participants