Skip to content

Debugger improvement part 3#6789

Merged
yann300 merged 20 commits intomasterfrom
debugger_v3
Feb 11, 2026
Merged

Debugger improvement part 3#6789
yann300 merged 20 commits intomasterfrom
debugger_v3

Conversation

@yann300
Copy link
Contributor

@yann300 yann300 commented Feb 9, 2026

No description provided.

@netlify
Copy link

netlify bot commented Feb 9, 2026

Deploy Preview for reliable-cocada-166884 ready!

Name Link
🔨 Latest commit f6c934f
🔍 Latest deploy log https://app.netlify.com/projects/reliable-cocada-166884/deploys/698c5f7a44a800000868d6d6
😎 Deploy Preview https://deploy-preview-6789--reliable-cocada-166884.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link

nx-cloud bot commented Feb 9, 2026

View your CI Pipeline Execution ↗ for commit f6c934f

Command Status Duration Result
nx lint vyper ✅ Succeeded <1s View ↗
nx lint remixd ✅ Succeeded <1s View ↗
nx lint solhint ✅ Succeeded <1s View ↗
nx lint doc-gen ✅ Succeeded <1s View ↗
nx lint remix-ui ✅ Succeeded 14s View ↗
nx lint learneth ✅ Succeeded <1s View ↗
nx lint debugger ✅ Succeeded <1s View ↗
nx lint remix-lib ✅ Succeeded <1s View ↗
Additional runs (23) ✅ Succeeded ... View ↗

☁️ Nx Cloud last updated this comment at 2026-02-11 11:01:42 UTC

@ci-pr-comment
Copy link

ci-pr-comment bot commented Feb 9, 2026

✅ E2E tests passed (workflow: web)

Last run: Wed, Feb 11, 2026, 11:11 AM UTC

All tests are now passing! Previous failures have been resolved.

Copy link
Collaborator

@Aniket-Engg Aniket-Engg left a comment

Choose a reason for hiding this comment

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

CHeck and remove all unnecessary commented code and logs

runs = parseInt(data.result[0].Runs)
} catch (e) { }

/*
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove commented code if not required anymore

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I would like to keep that one, just as a reminder


targetPath = `${targetPath}/${network.id}/${contractAddress}`
let data
/*
Copy link
Collaborator

Choose a reason for hiding this comment

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

here too

Copy link
Contributor Author

Choose a reason for hiding this comment

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

same here

@yann300 yann300 enabled auto-merge February 11, 2026 10:53
@yann300 yann300 merged commit 67f72ba into master Feb 11, 2026
34 checks passed
@yann300 yann300 deleted the debugger_v3 branch February 11, 2026 11:27
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.

2 participants