Skip to content

feat: add Replace and SetByPath methods to Node for in-place updates#956

Draft
TheApeMachine wants to merge 1 commit into
bytedance:mainfrom
TheApeMachine:feat/ast-node-replace-setby-path
Draft

feat: add Replace and SetByPath methods to Node for in-place updates#956
TheApeMachine wants to merge 1 commit into
bytedance:mainfrom
TheApeMachine:feat/ast-node-replace-setby-path

Conversation

@TheApeMachine

Copy link
Copy Markdown

What type of PR is this?

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov-commenter

codecov-commenter commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 30.83333% with 83 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.10%. Comparing base (59be92f) to head (1a8b66c).
⚠️ Report is 83 commits behind head on main.

Files with missing lines Patch % Lines
ast/node.go 30.83% 59 Missing and 24 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #956      +/-   ##
==========================================
- Coverage   51.86%   51.10%   -0.77%     
==========================================
  Files         127      170      +43     
  Lines       10893    14265    +3372     
==========================================
+ Hits         5650     7290    +1640     
- Misses       4920     6561    +1641     
- Partials      323      414      +91     
Flag Coverage Δ
arm 42.76% <30.83%> (?)
macos-latest 43.94% <35.23%> (?)
ubuntu-24.04-arm 42.74% <30.83%> (?)
ubuntu-latest 49.62% <35.23%> (?)
x86 49.62% <35.23%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

3 participants