Skip to content

chore: update reviewer list - #4200

Merged
ianna merged 8 commits into
scikit-hep:mainfrom
pfackeldey:rm_myself
Jul 18, 2026
Merged

chore: update reviewer list#4200
ianna merged 8 commits into
scikit-hep:mainfrom
pfackeldey:rm_myself

Conversation

@pfackeldey

@pfackeldey pfackeldey commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator
  • Removed Peter Fackeldey and Iason Krommydas as regular reviewers on their request
  • Add Tai Sakuma as a regular reviewer

@github-actions github-actions Bot added the type/chore PR title type: chore (set automatically) label Jul 10, 2026
@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.97%. Comparing base (d14aed1) to head (1b14717).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@pfackeldey pfackeldey changed the title chore: rm myself chore: rm myself from reviewer list Jul 10, 2026
ianna
ianna previously approved these changes Jul 10, 2026
@henryiii

Copy link
Copy Markdown
Member

Removing a review lowers code coverage? Hmm, I guess it does, fewer eyes to look at code. :)

@ianna

ianna commented Jul 10, 2026

Copy link
Copy Markdown
Member

the tests failure is unrelated and fixed in #4198

@ianna ianna changed the title chore: rm myself from reviewer list chore: update reviewer list Jul 10, 2026
@github-actions

Copy link
Copy Markdown

The documentation preview is ready to be viewed at http://preview.awkward-array.org.s3-website.us-east-1.amazonaws.com/PR4200

@ianna
ianna enabled auto-merge (squash) July 18, 2026 00:32
@ianna
ianna merged commit 4446d42 into scikit-hep:main Jul 18, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/chore PR title type: chore (set automatically)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants