-
Notifications
You must be signed in to change notification settings - Fork 2
Completed Readme Documentation based on the directories #7
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
base: main
Are you sure you want to change the base?
Conversation
lowpolyneko
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello, thanks for contributing! Please take a look at my feedback when you have a chance.
| You can install Build Pulse directly from the source using `cargo`. Make sure you have the Rust toolchain installed. | ||
|
|
||
| ```bash | ||
| cargo install --git [https://github.com/Akhilanandateja/build-pulse](https://github.com/Akhilanandateja/build-pulse) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The correct url is https://github.com/lowpolyneko/build-pulse
| # Build Pulse 🦀 | ||
|
|
||
| a research-focused Jenkins CI/CD static build analyzer | ||
| A tiny, fast command-line utility that watches for file changes in a directory and automatically runs a specified command. It's a perfect companion for your development workflow, automating tasks like compiling code, running tests, or refreshing a server. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The description is incorrect, this project statically analyzes build artifacts from CI/CD pipelines and generates an easily embedded HTML report and corresponding database. I'd suggest to build the project with cargo doc and read the fairly well documented source code.
|
|
||
| ## Installation | ||
|
|
||
| You can install Build Pulse directly from the source using `cargo`. Make sure you have the Rust toolchain installed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some further instructions would be beneficial. (i.e. "What cargo commands should I run?")
|
|
||
| ## Why Use Build Pulse? | ||
|
|
||
| * **⚡ Blazingly Fast**: Built in Rust for minimal resource usage and instant response times. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: I personally dislike emojis, if they could be removed I'd appreciate that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, these bullets seem clearly AI generated, I would appreciate if you could spend some time and write more thoughtful bullets.
No description provided.