Skip to content

Added OpenMS Inc. & removed executive committee info#223

Merged
tjeerdijk merged 3 commits intoOpenMS:mainfrom
matteopilz:main
Oct 30, 2025
Merged

Added OpenMS Inc. & removed executive committee info#223
tjeerdijk merged 3 commits intoOpenMS:mainfrom
matteopilz:main

Conversation

@matteopilz
Copy link
Copy Markdown
Contributor

@matteopilz matteopilz commented Oct 30, 2025

Brief description of what is fixed or changed

Summary by CodeRabbit

  • Documentation
    • About page header updated to "OpenMS Inc."
    • Governance content simplified: replaced the previous Executive Committee details with a board-of-directors statement naming Dr. Samuel Wein (CEO), Prof. Dr. Oliver Kohlbacher (CSO), and Dr. Timo Sachsenberg (CTO)
    • Removed detailed committee responsibilities, meeting/election rules, and core-developer governance text
    • Site display text updated to use "OpenMS Inc." across navigation, footer, and homepage content

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 30, 2025

Deploy Preview for openms ready!

Name Link
🔨 Latest commit 0d1278a
🔍 Latest deploy log https://app.netlify.com/projects/openms/deploys/69034b0cd6ac3d000842616c
😎 Deploy Preview https://deploy-preview-223--openms.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Oct 30, 2025

Walkthrough

Replaces the detailed "Executive Committee" governance section in content/en/about.md with a concise "OpenMS Inc." board-of-directors statement, and updates public-facing strings in config.yaml to use "OpenMS Inc." instead of "OpenMS Project."

Changes

Cohort / File(s) Summary
About page governance update
content/en/about.md
Removed the ## Executive Committee section and all prior governance/bylaws text; added ## OpenMS Inc. with a single board-of-directors declaration naming Dr. Samuel Wein (chief executive officer), Prof. Dr. Oliver Kohlbacher (chief scientific officer), and Dr. Timo Sachsenberg (chief technology officer).
Site copy updates
config.yaml
Replaced display strings referencing "OpenMS Project" with "OpenMS Inc." across navbar/footer/hero/logo entries and adjusted YAML indentation/formatting for lists (no logic changes).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Mostly content/text updates across 1–2 files.
  • Review focus: correctness of names/titles, that no governance/legal text was inadvertently removed beyond intended scope, and YAML indentation preserved.

Suggested reviewers

  • jpfeuffer
  • tjeerdijk

Poem

🐰 I hopped through lines of bylaws and fuss,
I nibbled old clauses without much muss.
Now three names gleam on the page so bright,
OpenMS Inc. steps into the light. ✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The PR description is incomplete and contains only the template placeholder text with no actual content provided by the author. The template clearly specifies a requirement for a "Brief description of what is fixed or changed," either by referencing an issue or describing the changes, but the author has left only the template comments without filling in any meaningful description, implementation details, or issue references. The author should replace the template placeholder text with an actual description of the changes. This should include a brief explanation of why the governance structure was updated, the specific changes made (updating branding from "OpenMS Project" to "OpenMS Inc." and replacing the Executive Committee with board-of-directors information), and any relevant issue references using the "Fixes gh-NNNN" format if applicable.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (1 passed)
Check name Status Explanation
Title Check ✅ Passed The PR title "Added OpenMS Inc. & removed executive committee info" accurately reflects the primary changes in the changeset. The summary confirms that the changes include updating references from "OpenMS Project" to "OpenMS Inc." in multiple files (config.yaml and about.md header) and replacing the Executive Committee governance section with a board-of-directors declaration. The title is concise, specific, and clearly communicates the two main aspects of the changeset without unnecessary noise or vague terminology.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2d348da and 0d1278a.

📒 Files selected for processing (2)
  • config.yaml (6 hunks)
  • content/en/about.md (1 hunks)
🔇 Additional comments (3)
config.yaml (1)

147-147: Branding updates look good.

The "OpenMS Project" → "OpenMS Inc." text replacements at footer (line 147) and navbar (line 182) align with the PR objectives and are consistent with the rebranding effort in the parallel content/en/about.md changes.

Also applies to: 182-182

content/en/about.md (2)

2-2: Corporate branding and board structure established correctly.

The page title (line 2) and board-of-directors statement (line 8) accurately reflect the OpenMS Inc. entity and leadership structure. The capitalization is consistent ("OpenMS Inc." used throughout).

Also applies to: 8-8


10-26: Retention of community sections is appropriate.

The Sponsors, Institutional Partners, and Donate sections remain intact, which is correct as these do not relate to the Executive Committee removal and align with the PR's objective to replace governance detail with a concise board statement.


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.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
content/en/about.md (1)

14-14: Simplify the paragraph and avoid redundancy.

The 501(c)(3) nonprofit status is mentioned twice in quick succession ("nonprofit organization" and "As a registered 501(c)(3) nonprofit organization"), making the paragraph feel repetitive. Consider restructuring to reduce redundancy and improve clarity, particularly around the transition between the mission statement and the legal notice.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 6fa51a2 and be0292c.

📒 Files selected for processing (1)
  • content/en/about.md (1 hunks)

Comment thread content/en/about.md Outdated
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

♻️ Duplicate comments (1)
content/en/about.md (1)

16-16: Capitalization inconsistency resolved.

The past review flagged inconsistent capitalization of "OpenMS Inc." on this line. The current version correctly uses "OpenMS Inc." with proper capitalization and punctuation, consistent with the section header. ✓

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between be0292c and 2d348da.

📒 Files selected for processing (1)
  • content/en/about.md (1 hunks)

Comment thread content/en/about.md
Copy link
Copy Markdown
Contributor

@tjeerdijk tjeerdijk left a comment

Choose a reason for hiding this comment

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

Looks good

@tjeerdijk tjeerdijk merged commit 957739b into OpenMS:main Oct 30, 2025
5 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Oct 31, 2025
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.

2 participants