-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
docs(community): update latest tsc members list #4494
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
Closed
asyncapi-bot
wants to merge
1
commit into
master
from
update-ambassadors-56521b8e4eb176d781ab053d504f38e9ec3a36f1
Closed
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
Missing required fields for TSC member entry.
Given the PR objective to "update latest tsc members list," this entry is missing:
"bio"field (required in all other entries)"isTscMember"flag (other TSC members like Daniel Kocot, Hugo Guerrero, Hari Krishnan, and Naresh Jain have"isTscMember": true)Compare the structure against existing TSC members (e.g., line 65 for Daniel Kocot) to ensure consistency and completeness.
Is Zbigniew Malcherczyk being added as a TSC member? If so, add the missing
"isTscMember": truefield and a"bio"field to match other entries.I'll verify the claims in the review comment by examining the JSON file structure.
Length of output: 1317
🏁 Script executed:
Length of output: 500
🏁 Script executed:
Length of output: 98
🏁 Script executed:
Length of output: 1354
Add missing
biofield and clarify TSC member status.The entry is missing two standard fields present in all other TSC members:
"bio"field (present in 12 of 13 entries; Zbigniew is the only exception)"isTscMember": trueflag (present in all four existing TSC members: Daniel Kocot, Hugo Guerrero, Hari Krishnan, Naresh Jain)If Zbigniew is being added as a TSC member (as the PR description suggests), add both the
"bio"and"isTscMember": truefields to match the pattern of other TSC members.🤖 Prompt for AI Agents
🧩 Analysis chain
Verify the duplicate entry claim and validate JSON integrity.
The AI summary states the Zbigniew Malcherczyk entry appears twice, but the provided code shows only one instance (lines 755-803). Verify the file integrity and check whether a duplicate actually exists.
Additionally, validate the JSON structure and count ambassador entries:
🏁 Script executed:
Length of output: 134
Now let me verify the entry's structure and check if it has required fields:
🏁 Script executed:
Length of output: 4098
Remove the incorrect duplicate claim and address missing data fields.
The verification confirms there is NO duplicate entry—only one Zbigniew Malcherczyk entry exists at index 11. However, the entry is missing fields that are present in 12 out of 13 other ambassador entries:
biofield (12/13 entries have this)slackfield (present in most other entries)Additionally, verify whether this entry should include
"isTscMember": true, as the PR title references "update latest tsc members list" but currently only 4 of the 13 entries have this flag set.🤖 Prompt for AI Agents