rdubois-crypto / FreshCryptoLib

Cryptographic Primitives for Blockchain Systems (solidity, cairo, C and rust)
MIT License
124 stars 22 forks source link

🎨 Improve WebAuthn Forge development workflow #3

Closed qd-qd closed 1 year ago

qd-qd commented 1 year ago

This pull request improves the WebAuthn Forge development workflow by adding a Makefile with targets for building, testing, and linting the project. The build target runs forge build, the test target runs forge test --ffi, and the lint target runs forge fmt --check. Additionally, the clean target runs forge clean to remove build artifacts, and the install target runs git submodule update --init to install dependencies. These targets make it easier to perform common development tasks and ensure code quality.

This pull request also includes improvements to the .gitignore file and the removal of the paper submodule. The tests have been rewritten to remove file-system operations, improving performance and reducing complexity. The GitHub Actions workflow runs the build command, the tests, and the linter. Finally, some comments were removed from the forge tests, and lint commands were added to ensure code quality.

rdubois-crypto commented 1 year ago

lgtm