Skip to content
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

Enable release of v0.8.0-alpha.4 #403

Merged
merged 14 commits into from Aug 23, 2023
Merged

Enable release of v0.8.0-alpha.4 #403

merged 14 commits into from Aug 23, 2023

Conversation

FabijanC
Copy link
Collaborator

@FabijanC FabijanC commented Aug 11, 2023

Usage related changes

  • Support --single-file flag of starknet-compile
  • Support REVERTED status of transactions
  • Change output of failed transactions: just stringify the status object not to rely on its properties (e.g. tx_failure_reason had been removed)
  • Remove constructor checks - assuming cairo compiler has them by now (it didn't use to, e.g. double constructor wasn't checked)

Development related changes

  • Update Devnet (0.6.0) - 0.5.* wasn't adapted to the new API - one test involving forking was failing
  • Update cairo compiler (2.1.0)

Checklist:

  • Formatted the code
  • No linter errors + tried to avoid introducing linter warnings
  • Performed a self-review of the code
  • Rebased to the last commit of the target branch (or merged it into my branch)
  • Documented the changes
  • Updated the test directory (with a test case consisting of network.json, hardhat.config.ts, check.ts)
  • Linked issues which this PR resolves
  • Created a PR to the plugin branch of starknet-hardhat-example:
  • All tests are passing (for external contributors who don't have access to the CI/CD pipeline)

@penovicp penovicp merged commit 195a59c into master Aug 23, 2023
6 checks passed
@penovicp penovicp deleted the fix-0.8.0-alpha.4 branch August 23, 2023 09:28
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.

None yet

2 participants