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(p/math_eval, gnovm): Simplifying Previous Error Formatting with Errorf #1772

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

notJoon
Copy link
Member

@notJoon notJoon commented Mar 14, 2024

Description

Replace previouse formatted error creations with fmt.Errof (in gno, ufmt.Errorf) to simplify error creation and formatting.

Follow up for PR #1767

@notJoon notJoon requested review from jaekwon, piux2, thehowl, moul and a team as code owners March 14, 2024 06:47
@github-actions github-actions bot added 🧾 package/realm Tag used for new Realms or Packages. 📦 🤖 gnovm Issues or PRs gnovm related labels Mar 14, 2024
Copy link

codecov bot commented Mar 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.51%. Comparing base (3481a03) to head (3f3e786).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1772      +/-   ##
==========================================
+ Coverage   44.81%   47.51%   +2.69%     
==========================================
  Files         459      388      -71     
  Lines       67707    61373    -6334     
==========================================
- Hits        30342    29159    -1183     
+ Misses      34827    29772    -5055     
+ Partials     2538     2442      -96     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@thehowl thehowl merged commit 486511f into gnolang:master Mar 14, 2024
190 of 191 checks passed
@notJoon notJoon deleted the use-errorf branch March 15, 2024 02:56
albttx pushed a commit to albttx/gno that referenced this pull request Mar 15, 2024
…rorf` (gnolang#1772)

## Description

Replace previouse formatted error creations with `fmt.Errof` (in gno,
`ufmt.Errorf`) to simplify error creation and formatting.

Follow up for PR gnolang#1767
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 🤖 gnovm Issues or PRs gnovm related 🧾 package/realm Tag used for new Realms or Packages.
Projects
Status: Done
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants