Skip to content

feat: Expose function name node#42

Merged
abhisek merged 1 commit intomainfrom
feat/expose-function-name-node
Aug 6, 2025
Merged

feat: Expose function name node#42
abhisek merged 1 commit intomainfrom
feat/expose-function-name-node

Conversation

@abhisek
Copy link
Member

@abhisek abhisek commented Aug 6, 2025

No description provided.

@abhisek abhisek requested review from a team and Copilot August 6, 2025 16:50
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR exposes the internal function name node through a new getter method to provide external access to the underlying tree-sitter node representing the function name.

  • Adds a new public getter method GetFunctionNameNode() to the FunctionDeclarationNode struct

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 57.86%. Comparing base (91cb669) to head (20bddbd).

Files with missing lines Patch % Lines
core/ast/function_decl.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   57.82%   57.86%   +0.04%     
==========================================
  Files          47       47              
  Lines        4578     4580       +2     
==========================================
+ Hits         2647     2650       +3     
  Misses       1707     1707              
+ Partials      224      223       -1     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@abhisek abhisek enabled auto-merge (squash) August 6, 2025 16:53
@abhisek abhisek merged commit 6c4f6f1 into main Aug 6, 2025
5 checks passed
@abhisek abhisek deleted the feat/expose-function-name-node branch August 6, 2025 16:53
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.

4 participants