Skip to content

Recognize former security maintainers - #6567

Merged
cwperks merged 2 commits into
opensearch-project:mainfrom
cwperks:docs/restore-emeritus-maintainers
Sep 29, 2026
Merged

cwperks merged 2 commits into
opensearch-project:mainfrom
cwperks:docs/restore-emeritus-maintainers

Conversation

@cwperks

@cwperks cwperks commented Sep 29, 2026

Copy link
Copy Markdown
Member

Description

Restore recognition of the eight former maintainers removed from the maintainers list in #1607 by adding them to the existing Emeritus section:

  • afazel
  • Andy (andy840314)
  • hardik-k-shah
  • hsiang9431-amzn
  • lukkoor
  • Palash Hedau (palashhedau)
  • Vlad Rozov (vrozov)
  • vengadanathan-s

Preserve their names and GitHub links from that PR. Label the restored Amazon affiliations as being during their tenure rather than asserting current employment.

Also add a thank-you for emeritus maintainers' contributions and stewardship, and a table-of-contents link to the section. Existing current and emeritus maintainer entries are unchanged. This is recognition only and does not change active maintainer responsibilities or repository access.

Validation

  • Compared the restored entries with the diff in Update maintainers list #1607 and checked that none duplicate existing maintainers.
  • git diff --check.
  • ./gradlew :precommit.

Signed and signed off for DCO.

Signed-off-by: Craig Perkins <craig5008@gmail.com>
@github-actions

github-actions Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

(Review updated until commit c456c6f)

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

Signed-off-by: Craig Perkins <craig5008@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor

Persistent review updated to latest commit c456c6f

@cwperks
cwperks merged commit 8cc42bf into opensearch-project:main Sep 29, 2026
150 of 157 checks passed
@codecov

codecov Bot commented Sep 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.05%. Comparing base (c5cf7f0) to head (c456c6f).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6567      +/-   ##
==========================================
+ Coverage   76.01%   76.05%   +0.04%     
==========================================
  Files         467      470       +3     
  Lines       31175    31274      +99     
  Branches     4692     4709      +17     
==========================================
+ Hits        23697    23787      +90     
- Misses       5296     5298       +2     
- Partials     2182     2189       +7     

see 19 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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