Skip to content

add Fricounet as a committer#715

Merged
imeoer merged 1 commit intocontainerd:mainfrom
imeoer:maintainers
Mar 3, 2026
Merged

add Fricounet as a committer#715
imeoer merged 1 commit intocontainerd:mainfrom
imeoer:maintainers

Conversation

@imeoer
Copy link
Collaborator

@imeoer imeoer commented Mar 3, 2026

Baptiste Girard-Carrabin @Fricounet works at DataDog and is a core contributor of the nydus project, he has contributed valuable fixups / features to the nydus-snapshotter and nydus repos, it would be great to add him as a committer for the nydus-snapshotter project.

Needs explicit LGTM from 2/3 of the nydus-snapshotter committers:

@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 22.59%. Comparing base (fc330cc) to head (fa82d37).
⚠️ Report is 13 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #715      +/-   ##
==========================================
+ Coverage   22.02%   22.59%   +0.56%     
==========================================
  Files         130      131       +1     
  Lines       11931    12018      +87     
==========================================
+ Hits         2628     2715      +87     
+ Misses       8960     8957       -3     
- Partials      343      346       +3     

see 9 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@changweige
Copy link
Member

LGTM

@imeoer imeoer merged commit 2929b11 into containerd:main Mar 3, 2026
17 checks passed
@imeoer
Copy link
Collaborator Author

imeoer commented Mar 13, 2026

Hi @dmcgowan , could you grant @Fricounet and @BraveY access to the containerd/nydus-snapshotter repository?

Specifically, @Fricounet as a Committer, and @BraveY as a Reviewer (as mentioned in #646).

Thanks so much!

@dmcgowan
Copy link
Member

Done

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