Skip to content

Conversation

@miguelcsx
Copy link
Contributor

Summary

This pull request introduces significant updates across multiple components of the project:

  1. rust-code-analysis-web:
    Added a README.md with comprehensive documentation on how to build and run the web server. This includes instructions for setting up the project, dependencies, and starting the server.

  2. rust-code-analysis-cli:
    Updated the project with a new README.md file providing detailed instructions for building and running the command-line interface (CLI). The documentation includes usage examples and options available in the CLI.

Benefits

  • Improved documentation for easier setup and usage of rust-code-analysis-web and rust-code-analysis-cli.

@miguelcsx
Copy link
Contributor Author

@marco-c This first PR includes readmes for developers, about the flags that have been created, the structure of the project, and how to build and run, let me know any feedback, thanks.

@miguelcsx
Copy link
Contributor Author

@marco-c I made the suggested changes, I look forward to any comments.

@Luni-4 Luni-4 requested a review from marco-c March 31, 2025 11:54
Copy link
Collaborator

@Luni-4 Luni-4 left a comment

Choose a reason for hiding this comment

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

@miguelcsx

Thanks a lot for your work! This is fine for me! @marco-c, if you agree, we can merge

- added documentation per package module

- included information about the code, how to build and run it

- give information about the flags that can be used
- reworded the first sentences to make it clearer

- removed project structure as requested

- removed sentence about rust installation

- replaced `cargo run` with respective command
@marco-c marco-c merged commit b2daee1 into mozilla:master May 5, 2025
1 check passed
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.

3 participants