Skip to content

Readme Updated #93

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

Closed
wants to merge 14 commits into from
Closed

Conversation

AleksaZatezalo
Copy link
Contributor

What has been added to README.md includes:

  1. A description of the folder structure, and code structure via UML Diagrams, and a short description of the corresponding classes.
  2. A description of how to run testcases before opening a pull request.
  3. Generic guidelines on coding style.

@AleksaZatezalo
Copy link
Contributor Author

Wrong thing in readme.

@AleksaZatezalo
Copy link
Contributor Author

Wrong instruction regarding test bench.

alessandromaggio added a commit that referenced this pull request Oct 25, 2022
Allow count greater than 16-bit, fix pylint issues, update and expand readme
alessandromaggio added a commit that referenced this pull request Oct 25, 2022
* fix(icmp.py): ICMP header format string

All values in the struct should be unsigned.

* pylint issues resolved

Fixed a bunch of pylint issues

* First Readme push headers updated.

* Update README.md

* Update README.md

* UMLs created.

* UMLs created.

* Folder Structure Created.

* backticks added.

* Supporting words added.

* Grammer fix

* Code Styles section finished.

* Fix made

* Fix made

* Readme fixed.

* Readme fixed.

* Ready for pull request.

* Finally ready for pull request.

* Testcase added,

* feat: integrate #89, #92, and #93

Allow count greater than 16-bit, fix pylint issues, update and expand readme

Co-authored-by: David A. Bell <8835326+dabell-cc@users.noreply.github.com>
Co-authored-by: Mark Mayo <mark@there.co.nz>
Co-authored-by: Aleksa Zatezalo <aleksa@kali.azatezalo>
Co-authored-by: Aleksa Zatezalo <59803757+AleksaZatezalo@users.noreply.github.com>
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.

1 participant