Skip to content

chore: update mentor roster with new triage members#1732

Draft
Pradeepbeleri wants to merge 1 commit intohiero-ledger:mainfrom
Pradeepbeleri:chore/update-mentor-roster
Draft

chore: update mentor roster with new triage members#1732
Pradeepbeleri wants to merge 1 commit intohiero-ledger:mainfrom
Pradeepbeleri:chore/update-mentor-roster

Conversation

@Pradeepbeleri
Copy link
Contributor

Fixes #1693

Updated .github/mentor_roster.json with the new triage members as requested.

Signed-off-by: Pradeepbeleri <pradpie@gmail.com>
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

Walkthrough

The PR adds six new members (Mounil2005, parvninama, drtoxic69, prajeeta15, undefinedIsMyLife, cheese-cakee) to the mentor roster in .github/mentor_roster.json and documents this addition in the CHANGELOG under the Unreleased section.

Changes

Cohort / File(s) Summary
Mentor Roster Update
.github/mentor_roster.json
Added six new members to the order array in the mentor roster configuration.
Changelog Entry
CHANGELOG.md
Documented the mentor roster update in the Unreleased -> Added section.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating the mentor roster with new triage members, which matches the changeset.
Description check ✅ Passed The description is directly related to the changeset, mentioning the issue fix and the roster update made.
Linked Issues check ✅ Passed The PR successfully adds all six required usernames (Mounil2005, parvninama, drtoxic69, prajeeta15, undefinedIsMyLife, cheese-cakee) to the mentor roster and includes a CHANGELOG entry as required by issue #1693.
Out of Scope Changes check ✅ Passed The PR modifies only .github/mentor_roster.json and CHANGELOG.md, both of which are within scope per issue #1693 requirements.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@exploreriii exploreriii added the status: needs triage review PR needs a review from the triage team label Feb 6, 2026
@exploreriii
Copy link
Contributor

Request review if available @hiero-ledger/hiero-sdk-python-triage

Copy link
Contributor

@tech0priyanshu tech0priyanshu left a comment

Choose a reason for hiding this comment

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

LGTM @Pradeepbeleri Thanks for contributing

Copy link
Member

@AntonioCeppellini AntonioCeppellini left a comment

Choose a reason for hiding this comment

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

Nice Work! :D thank you for your contribution!

@Pradeepbeleri
Copy link
Contributor Author

Thanks @tech0priyanshu and @AntonioCeppellini for the review.

@exploreriii exploreriii added status: awaiting merge and removed status: needs triage review PR needs a review from the triage team labels Feb 6, 2026
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1732   +/-   ##
=======================================
  Coverage   93.22%   93.22%           
=======================================
  Files         140      140           
  Lines        9017     9017           
=======================================
  Hits         8406     8406           
  Misses        611      611           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Hi, this is MergeConflictBot.
Your pull request cannot be merged because it contains merge conflicts.

Please resolve these conflicts locally and push the changes.

Quick Fix for CHANGELOG.md Conflicts

If your conflict is only in CHANGELOG.md, you can resolve it easily using the GitHub web editor:

  1. Click on the "Resolve conflicts" button in the PR
  2. Accept both changes (keep both changelog entries)
  3. Click "Mark as resolved"
  4. Commit the merge

For all other merge conflicts, please read:

Thank you for contributing!

@tech0priyanshu
Copy link
Contributor

Hi @Pradeepbeleri, please resolve the merge conflicts. Sorry for the inconvenience—everything else in the PR looks fine. Please refer to Resolving Merge Conflicts Rebasing Guide

@tech0priyanshu tech0priyanshu added status: needs developer revision PR has requested changes that the developer needs to implement and removed status: awaiting merge labels Feb 6, 2026
@exploreriii exploreriii marked this pull request as draft February 7, 2026 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: needs developer revision PR has requested changes that the developer needs to implement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Good First Issue]: Update mentor roster with new triage members

5 participants