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

chore: ink v5.1 #409

Merged
merged 2 commits into from
Dec 19, 2024
Merged

chore: ink v5.1 #409

merged 2 commits into from
Dec 19, 2024

Conversation

Daanvdplas
Copy link
Collaborator

@Daanvdplas Daanvdplas commented Dec 15, 2024

In v5.0 there was an ugly warning from the linter which is fixed in v5.1. Also good to uplift the version.

@codecov-commenter
Copy link

codecov-commenter commented Dec 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.41%. Comparing base (3476994) to head (0c431cf).

@@           Coverage Diff           @@
##             main     #409   +/-   ##
=======================================
  Coverage   68.41%   68.41%           
=======================================
  Files          70       70           
  Lines       11838    11838           
  Branches    11838    11838           
=======================================
  Hits         8099     8099           
  Misses       3482     3482           
  Partials      257      257           

@chungquantin
Copy link
Collaborator

Should we use git path to latest main branch then?

@Daanvdplas
Copy link
Collaborator Author

Should we use git path to latest main branch then?

Why?

@chungquantin
Copy link
Collaborator

Should we use git path to latest main branch then?

Why?

To avoid updating the version every time there's a new update. Anyway, this could brings unexpected issues so ignore my comment.

Copy link
Collaborator

@chungquantin chungquantin left a comment

Choose a reason for hiding this comment

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

LGTM! Contracts compiled and built.

pop-api/src/v0/fungibles/errors.rs Show resolved Hide resolved
@Daanvdplas Daanvdplas requested a review from AlexD10S December 19, 2024 14:32
Copy link
Collaborator

@AlexD10S AlexD10S left a comment

Choose a reason for hiding this comment

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

Everything looks great! I tested deploying a smart contract and interacting with it, and everything is working perfectly.

@Daanvdplas Daanvdplas merged commit c6ad126 into main Dec 19, 2024
13 checks passed
@Daanvdplas Daanvdplas deleted the daan/chore-ink_v5.1 branch December 19, 2024 15:33
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.

4 participants