Skip to content

Conversation

ej2432
Copy link
Member

@ej2432 ej2432 commented Aug 14, 2025

adds a link to the governance documentation on github. one question - only advisory committee members and pm are named on this page. should others be added here?

adds a link to the governance documentation on github. one question - only advisory committee members and pm are named on this page. should others be added here?
@ej2432 ej2432 requested review from magdmartin and Ainali August 14, 2025 16:21
Copy link

netlify bot commented Aug 14, 2025

Deploy Preview for openrefine-website failed.

Name Link
🔨 Latest commit fc0d11b
🔍 Latest deploy log https://app.netlify.com/projects/openrefine-website/deploys/689e0d298c1a130008c056dd

Copy link
Member

@magdmartin magdmartin left a comment

Choose a reason for hiding this comment

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

Thanks for the PR.

In the governance document, we are naming members of the following groups

  • Core Dev Group
  • Release Manager
  • Advisory Committee
  • Project Manager

We are only missing the Code of Conduct Committee member list. They are listed in the Code of Conduct document

@@ -35,6 +35,9 @@ import Card from '../card.tsx';
<img src={OpenRefineIsMagicImageUrl} alt="OpenRefine is magic" id="magic"/>
<p style={{fontSize: '0.7em'}}><a href="https://commons.wikimedia.org/wiki/File:Open_Refine_is_Magic.png">CC-BY-SA Scott Carlson</a></p>
</div>
# Governance Model & Team Members
Learn more [on GitHub]([url](https://github.com/OpenRefine/OpenRefine/blob/master/GOVERNANCE.md)).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Learn more [on GitHub]([url](https://github.com/OpenRefine/OpenRefine/blob/master/GOVERNANCE.md)).
Review [OpenRefine Governance on GitHub](https://github.com/OpenRefine/OpenRefine/blob/master/GOVERNANCE.md).

Copy link
Member

Choose a reason for hiding this comment

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

I made the anchor explicit on the link. I also fixed the syntax

@@ -35,6 +35,9 @@ import Card from '../card.tsx';
<img src={OpenRefineIsMagicImageUrl} alt="OpenRefine is magic" id="magic"/>
<p style={{fontSize: '0.7em'}}><a href="https://commons.wikimedia.org/wiki/File:Open_Refine_is_Magic.png">CC-BY-SA Scott Carlson</a></p>
</div>
# Governance Model & Team Members
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
# Governance Model & Team Members
# Governance Model & Team Members

Copy link
Member

Choose a reason for hiding this comment

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

I think we need to add a line break before and after each header

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