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

Address versioning and presets issues #879

Merged
merged 5 commits into from Jan 30, 2024
Merged

Address versioning and presets issues #879

merged 5 commits into from Jan 30, 2024

Conversation

martriay
Copy link
Contributor

@martriay martriay commented Jan 30, 2024

Fixes #872, #873, #876. It also fixes:

  • missing class hashes
  • wrong cairo compiler statement in docs

@martriay martriay changed the title Fix cairo version in preset docs Fix version issues Jan 30, 2024
@martriay martriay changed the title Fix version issues Address versioning and presets issues Jan 30, 2024
Copy link
Member

@ericnordelo ericnordelo left a comment

Choose a reason for hiding this comment

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

LGTM!

On the other hand, shouldn't we create a PR for updating the release-v0.8.1 branch for just updating the wording? I still think we shouldn't wait next release for this. This is regarding #873

Copy link
Collaborator

@andrew-fleming andrew-fleming left a comment

Choose a reason for hiding this comment

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

LGTM!

@martriay martriay merged commit 1448616 into main Jan 30, 2024
5 checks passed
@martriay martriay deleted the fix-presets-docs branch January 30, 2024 16:55
@martriay
Copy link
Contributor Author

@ericnordelo created #880 for that

martriay added a commit that referenced this pull request Jan 31, 2024
* fix cairo version

* update class hashes

* fix scarb.toml version number

* add presets explainer

* update wording
martriay added a commit that referenced this pull request Jan 31, 2024
* Address versioning and presets issues (#879)

* fix cairo version

* update class hashes

* fix scarb.toml version number

* add presets explainer

* update wording

* fix wording

* fix word
martriay added a commit that referenced this pull request Jan 31, 2024
* bump version and update changelog

* fix docs links

* update class hashes to cairo 2.4.1

* Address versioning and presets issues (#879)

* fix cairo version

* update class hashes

* fix scarb.toml version number

* add presets explainer

* update wording

* fix wording

* fix word
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants