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 Error Message Bug - Add User Friendly String Representation for NodeKind #43935

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

krutip7
Copy link

@krutip7 krutip7 commented Mar 22, 2025

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Approach

Describe how you are implementing the solutions along with the design details.

  • Enhanced the NodeKind enum with custom string representation for user friendly messages

Samples

Provide high-level details about the samples related to this feature.

  • Here is a sample image of the existing bug (same as attached to the issue)

image

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

  • While the changes in this PR have been limited to the scope of the issue (i.e. the RESOURCE_FUNC enum), it could be extended further to include custom labels for the other enums as well.

Check List

  • Read the Contributing Guide
  • Updated Change Log
  • Checked Tooling Support (#)
  • Added necessary tests
    • Unit Tests
    • Spec Conformance Tests
    • Integration Tests
    • Ballerina By Example Tests
  • Increased Test Coverage
  • Added necessary documentation
    • API documentation
    • Module documentation in Module.md files
    • Ballerina By Examples

- Enhances NodeKind enum with custom string representation for user friendly messages
- Fixes the generated error message by replacing the keyword "resource_func" with a more user friendly string "resource function"
@CLAassistant
Copy link

CLAassistant commented Mar 22, 2025

CLA assistant check
All committers have signed the CLA.

@krutip7 krutip7 changed the title Enhance Error Messages - Add User Friendly String Representation for NodeKind Fix Error Message Bug - Add User Friendly String Representation for NodeKind Mar 22, 2025
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.

[Bug]: Invalid error message: must be resource function instead of resource_func
2 participants