Skip to content

Conversation

@Bennett-Petzold
Copy link
Collaborator

  • Full tests for all parsing (ignoring the instruction enum, as the only thing to test is switch case correctness)
  • Tests for the TRAP instruction parsing

GitHub actions more or less copied from my backed_data repository.

  • Dependabot (to get notifications about major dependency updates or vulnerabilities)
  • Full CI tests on all major architecture/OS pairs on GitHUb
  • MSRV CI test
  • CI test for compilation under all features
  • Auto release tagging when Cargo.toml version is bumped (will require a token setup on the wpkelso account)
  • Coverage testing
  • Clippy CI analysis (will only work once the repo is public)
  • Pages deploy for docs and coverage (will require a pages setup on the wpkelso account)
  • Badges to link to the above in README

Bennett-Petzold and others added 4 commits December 19, 2024 16:19
* Define word parses for each instruction

* Reorganize instructions to a file-per

* Fix scoping with tests

* Move instruction utils to a dedicated file

* Create full-capture instruction enum

* Add test cases for iadd parsing

* Fix jump test

* Fix add test ranges

* Add AND tests

* Add NOT test
Badges, tests on all architectures, automatic version tagging, clippy, docs
build, code coverage.
@github-actions
Copy link

Code Coverage Report: Only Changed Files listed

Code Coverage

Package Coverage
Overall Coverage 🟢 76.54%

Minimum allowed coverage is 0%, this run produced 76.54%

Copy link
Owner

@wpkelso wpkelso left a comment

Choose a reason for hiding this comment

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

I'll set the github actions up later

@wpkelso wpkelso merged commit a415ede into main Dec 20, 2024
8 of 9 checks passed
@Bennett-Petzold Bennett-Petzold deleted the full-parsing branch December 27, 2024 16:09
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