Skip to content

Comments

Add parameters lost during Java 21 migration#554

Merged
TharmiganK merged 3 commits intoballerina-platform:masterfrom
ravinperera00:fix_param
Jan 31, 2025
Merged

Add parameters lost during Java 21 migration#554
TharmiganK merged 3 commits intoballerina-platform:masterfrom
ravinperera00:fix_param

Conversation

@ravinperera00
Copy link
Contributor

Purpose

Examples

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility

@TharmiganK
Copy link
Contributor

Lets add a test case also by modifying this service:

service Service "filesystem" on localFolder {

service Service "filesystem" on localFolder {

    remote function onCreate(FileEvent m) {
        createInvoke = m.operation == "create";
    }

    remote function onModify(FileEvent m) {
        modifyInvoke = m.operation == "modify";
    }

    remote function onDelete(FileEvent m) {
        deleteInvoke = m.operation == "delete";
    }
}

@sonarqubecloud
Copy link

Copy link
Contributor

@TharmiganK TharmiganK left a comment

Choose a reason for hiding this comment

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

LGTM

@TharmiganK TharmiganK merged commit 36823b4 into ballerina-platform:master Jan 31, 2025
4 checks passed
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