Skip to content

Update ontoneo.md#2687

Merged
deepakunni3 merged 19 commits intoOBOFoundry:masterfrom
FernandaFarinelli:patch-3
Nov 3, 2025
Merged

Update ontoneo.md#2687
deepakunni3 merged 19 commits intoOBOFoundry:masterfrom
FernandaFarinelli:patch-3

Conversation

@FernandaFarinelli
Copy link
Copy Markdown
Contributor

No description provided.

@FernandaFarinelli
Copy link
Copy Markdown
Contributor Author

@matentzn Could you help me with this error?

@deepakunni3 deepakunni3 self-requested a review March 17, 2025 08:12
Copy link
Copy Markdown
Member

@deepakunni3 deepakunni3 left a comment

Choose a reason for hiding this comment

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

@FernandaFarinelli Thank you for updating the entry for Ontoneo.

After inspecting the errors, it looks like there are a couple of issues that needs to be fixed. These are provided via review comments.

Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
FernandaFarinelli and others added 3 commits March 17, 2025 18:39
Co-authored-by: Deepak <deepakunni3@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

@FernandaFarinelli FernandaFarinelli left a comment

Choose a reason for hiding this comment

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

I made some changes.

@deepakunni3 deepakunni3 self-requested a review April 25, 2025 08:42
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
@deepakunni3
Copy link
Copy Markdown
Member

deepakunni3 commented Apr 25, 2025

@FernandaFarinelli Thank you for incorporating the suggestions. Please go through and address the comments made in the re-review.

I also see another PR (#2686) that updates metadata for OntONeo. Is it possible to include those changes into this PR?

@nlharris
Copy link
Copy Markdown
Contributor

@FernandaFarinelli are you going to address the comments?

FernandaFarinelli and others added 3 commits October 23, 2025 21:41
Co-authored-by: Deepak <deepakunni3@users.noreply.github.com>
@FernandaFarinelli
Copy link
Copy Markdown
Contributor Author

@nlharris Please, could you help me with this? I tried to fix but it still with an error.

Copy link
Copy Markdown
Member

@deepakunni3 deepakunni3 left a comment

Choose a reason for hiding this comment

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

@FernandaFarinelli Thank you for incorporating the suggestions and changes.

Couple of observations upon re-review.

Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
Comment thread ontology/ontoneo.md Outdated
@deepakunni3
Copy link
Copy Markdown
Member

@FernandaFarinelli I also took the liberty to fix the errors you were seeing earlier. That was due to the metadata fields not being in the correct order. Fixed via 0c3fd83

@deepakunni3
Copy link
Copy Markdown
Member

@FernandaFarinelli Thank you for addressing the previous comments 👍

@deepakunni3 deepakunni3 self-requested a review November 3, 2025 15:40
@deepakunni3 deepakunni3 merged commit f7100b6 into OBOFoundry:master Nov 3, 2025
3 checks passed
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