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

fix(make): disable _test.gnoweb temporarily #1223

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

thehowl
Copy link
Member

@thehowl thehowl commented Oct 11, 2023

See-also: #1222

(h/t @jaekwon )

@thehowl thehowl requested a review from a team as a code owner October 11, 2023 00:18
@github-actions github-actions bot added 🧾 package/realm Tag used for new Realms or Packages. 📦 ⛰️ gno.land Issues or PRs gno.land package related labels Oct 11, 2023
@thehowl thehowl force-pushed the dev/morgan/disable-gnoweb branch 2 times, most recently from ccc023f to 3a47e33 Compare October 11, 2023 00:32
@github-actions github-actions bot removed the 🧾 package/realm Tag used for new Realms or Packages. label Oct 11, 2023
@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (fa8eb77) 47.86% compared to head (3a47e33) 47.80%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1223      +/-   ##
==========================================
- Coverage   47.86%   47.80%   -0.06%     
==========================================
  Files         369      369              
  Lines       62710    62710              
==========================================
- Hits        30018    29981      -37     
- Misses      30271    30304      +33     
- Partials     2421     2425       +4     

see 2 files with indirect coverage changes

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

@moul
Copy link
Member

moul commented Oct 11, 2023

Cc @gfanton, if you can quickly fix it, please close this PR. If you need more time for your in-memory PR, please merge this one first and then revert it in your PR.

Copy link
Member

@gfanton gfanton left a comment

Choose a reason for hiding this comment

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

@moul Im almost sure it will not be quick, even if Im able to finish it tomorrow it still need a review and a the PR will not be short.
So let merge this, I will try to open a draft PR ASAP with the in_memory node and a commit reverting this one

@gfanton gfanton merged commit dce345f into master Oct 11, 2023
177 of 242 checks passed
@gfanton gfanton deleted the dev/morgan/disable-gnoweb branch October 11, 2023 20:08
gfanton added a commit to gfanton/gno that referenced this pull request Oct 11, 2023
This reverts commit dce345f.

Signed-off-by: gfanton <8671905+gfanton@users.noreply.github.com>
thehowl added a commit to thehowl/gno that referenced this pull request Oct 21, 2023
gfanton pushed a commit to gfanton/gno that referenced this pull request Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 ⛰️ gno.land Issues or PRs gno.land package related
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants