Skip to content

point to Paul's new technical review instructions - #2956

Open
nlharris wants to merge 2 commits into
masterfrom
nlharris-patch-11
Open

nlharris wants to merge 2 commits into
masterfrom
nlharris-patch-11

Conversation

@nlharris

@nlharris nlharris commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

but note that this is not a NOR Manager responsibility.

but note that this is not a NOR Manager responsibility.
@nlharris
nlharris requested a review from pfabry October 2, 2026 16:25
@nlharris nlharris added documentation Issues related to documentation presented on the website or relevant to Foundry-provided tools attn: OFOC call Issue to discuss on fortnightly OBO Operations meeting labels Nov 2, 2026
@nlharris

nlharris commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

Need at least one review. @allenbaron ?

@allenbaron allenbaron left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think the technical review in the linked README and the one completed by an assigned reviewer are two distinct things.

The NOR README.md describes how to add an ontology to the NOR Dashboard and run lexical matching, so it does fall within the NOR manager's job description (at least from what I understand).

Maybe we could rename that section of the README.md so this overlap doesn't exist?

@nlharris

nlharris commented Nov 2, 2026

Copy link
Copy Markdown
Contributor Author

sure, add a suggestion.

@allenbaron

Copy link
Copy Markdown
Contributor

Thanks for @-ing me here. I was trying to find this info but couldn't seem to locate it.

My initial impression is that it would be helpful to migrate the information in the NOR README concerning who is responsible for what to the appropriate docs here, leaving only the technical how-to.

Let me look at these docs side-by-side in more detail and come up with a solid suggestion.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

attn: OFOC call Issue to discuss on fortnightly OBO Operations meeting documentation Issues related to documentation presented on the website or relevant to Foundry-provided tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants