Skip to content

Add li1 li2 extra fields#95

Closed
nanshi1177 wants to merge 5 commits into
masterfrom
add-li1-li2-extra-fields
Closed

Add li1 li2 extra fields#95
nanshi1177 wants to merge 5 commits into
masterfrom
add-li1-li2-extra-fields

Conversation

@nanshi1177

Copy link
Copy Markdown
Contributor

No description provided.

@nanshi1177 nanshi1177 requested a review from jmcclena April 1, 2026 18:15
@codecov

codecov Bot commented Apr 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.96%. Comparing base (3fe8166) to head (c58e029).

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #95   +/-   ##
=======================================
  Coverage   43.96%   43.96%           
=======================================
  Files          15       15           
  Lines       31419    31419           
=======================================
  Hits        13813    13813           
  Misses      17606    17606           

☔ View full report in Codecov by Sentry.
📢 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.

@nanshi1177 nanshi1177 closed this Apr 2, 2026
@nanshi1177 nanshi1177 deleted the add-li1-li2-extra-fields branch April 2, 2026 05:14
@smithsp

smithsp commented Apr 2, 2026

Copy link
Copy Markdown

Why delete this branch instead of merging?

@nanshi1177

Copy link
Copy Markdown
Contributor Author

Why delete this branch instead of merging?

The original PR accidentally included unrelated old commits from my local branch that were out of sync with remote master. I deleted it and recreated a clean PR with only the li_1/li_2 changes to keep the review simple. #97

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