search

RocksLabs / kvrocks-operator

Apache License 2.0
36 stars 15 forks source link

Add CI to check and linter the Go code style #7

Closed git-hulk closed 1 year ago

git-hulk commented 1 year ago

For more detail can see: https://github.com/KvrocksLabs/kvrocks_controller/blob/unstable/.github/workflows/ci.yaml

ColinChamber commented 1 year ago

OK, I will add it.

xiao-jay commented 1 year ago

@git-hulk hello, Can i try this issue?

xiao-jay commented 1 year ago

@git-hulk This project have roadmap or something? I don't know how to contribute.

git-hulk commented 1 year ago

@git-hulk hello, Can i try this issue?

Not sure if @ColinChamber has already on it.

@git-hulk This project have roadmap or something? I don't know how to contribute.

To see if @ColinChamber @jiayouxujin can create a roadmap for this project.

xiao-jay commented 1 year ago

OK, I will add it.

@ColinChamber I see you have assigned yourself, but you haven't mentioned PR yet. Can I participate in this issue？

ColinChamber commented 1 year ago

@xiao-jay Of course, everyone is welcome to participate

jiayouxujin commented 1 year ago

@xiao-jay Welcome to participate. I'm making e2e test work (because we're going to refactor some code, and I think making the test feasible first is a good solution). After testing it locally, I will split some issues. You are welcome to participate.

xiao-jay commented 1 year ago

@xiao-jay Welcome to participate. I'm making e2e test work (because we're going to refactor some code, and I think making the test feasible first is a good solution). After testing it locally, I will split some issues. You are welcome to participate.

ok,look forward to it.