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

Draft: add xgo trace tool (Please don't merge, just take a look and discuss) #11

Open
wants to merge 2 commits into
base: universe
Choose a base branch
from

Conversation

xhd2015
Copy link
Contributor

@xhd2015 xhd2015 commented Apr 15, 2024

This PR uses xgo tool trace to visualize the program

Produced by:

# install xgo
go install github.com/xhd2015/xgo/cmd/xgo@latest

cd test
xgo test -v ./
xgo tool trace TestEventStorePublish.json

I'm on the way understanding your internal logics, this trace helps.

image

@xhd2015
Copy link
Contributor Author

xhd2015 commented Apr 15, 2024

I'll try later to add more tests.

BTW, why you put all tests inside a test sub directory and even create a go.mod inside it? The go standard way is to place test files next to source files.

@xhd2015 xhd2015 changed the title Draft: add xgo trace tool Draft: add xgo trace tool (Please don't merge, just take a look and discuss) Apr 15, 2024
@DecentralizedInteligenceOrganization
Copy link
Collaborator

Thanks for contribution, i will review it later 💪

@DecentralizedInteligenceOrganization
Copy link
Collaborator

I need to set up contribution rules. Like using https://www.conventionalcommits.org/en/v1.0.0/

@xhd2015
Copy link
Contributor Author

xhd2015 commented Apr 18, 2024

Sure you should, we are procceding to a better way

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.

None yet

2 participants