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

feat: user-friendly command prefix #176

Closed
wants to merge 12 commits into from
Closed

feat: user-friendly command prefix #176

wants to merge 12 commits into from

Conversation

y1yang0
Copy link
Member

@y1yang0 y1yang0 commented Nov 12, 2024

related to #141

Add otelbuild prefix to go build to build your project:

$ otelbuild go build
$ otelbuild go build -o app cmd/app
$ otelbuild go build -gcflags="-m" cmd/app

The arguments for the tool itself should be placed before go build:

$ otelbuild -help                      # print help doc
$ otelbuild -debug go build            # enable debug mode
$ otelbuild -verbose go build          # print verbose log
$ otelbuild -rule=custom.json go build # use custom rule

@y1yang0
Copy link
Member Author

y1yang0 commented Nov 12, 2024

The phrase otelbuild go build is slightly redundant in terms of meaning (with two "build" words). Perhaps the tool name otelbuild could be renamed to something more meaningful. Are there any better suggestions? Looking forward to everyone's proposals.

@y1yang0
Copy link
Member Author

y1yang0 commented Nov 12, 2024

chatgpt says:

otel go build
otelpack go build
otelcompile go build
otelcraft go build
otelmaker go build
otelcreate go build
otelbuilder go build
otelcon go build
otelremix go build
inst go build

I prefer otelpack otel and otelremix

@123liuziming
Copy link
Collaborator

123liuziming commented Nov 12, 2024

We can discuss on today's meeting

@y1yang0
Copy link
Member Author

y1yang0 commented Nov 12, 2024

Make sense, let's keep it as it is now

@123liuziming
Copy link
Collaborator

prefer otelpack or otel

@y1yang0
Copy link
Member Author

y1yang0 commented Nov 13, 2024

Okay, let me change it to otel, and see if there are any objections or better suggestions

@y1yang0 y1yang0 closed this by deleting the head repository Nov 15, 2024
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.

2 participants