Skip to content

Conversation

@Caroline-99
Copy link
Contributor

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request updates version information in the pr_slim ontology files from 2025-11-13 to 2025-11-25. Despite the PR title claiming to add "collagen alpha-1(III) chain and collagen alpha-1(VI) chain to slim," the diffs only show version date updates with no actual protein additions or content changes.

Key Changes

  • Updated version dates from 2025-11-13 to 2025-11-25 in both OWL and OBO formats
  • No structural or content changes to the ontology files

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
pr_slim.owl Updated ontology version IRI and versionInfo annotation from 2025-11-13 to 2025-11-25
pr_slim.obo Updated data-version and owl:versionInfo property from 2025-11-13 to 2025-11-25

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +10 to +11
<http://purl.obolibrary.org/obo/pr/obophenotype/releases/2025-11-25/pr_slim.owl>
Annotation(owl:versionInfo "2025-11-25")
Copy link

Copilot AI Nov 25, 2025

Choose a reason for hiding this comment

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

The PR title states 'added collagen alpha-1(III) chain and collagen alpha-1(VI) chain to slim' but the changes only update version dates from 2025-11-13 to 2025-11-25. No actual collagen protein entries are being added in the diff. The PR title is misleading and should accurately reflect that this is only a version update, or the PR is missing the actual protein additions mentioned in the title.

Copilot uses AI. Check for mistakes.
@ar-ibrahim
Copy link
Contributor

@Caroline-99 Did you add these proteins to seed.txt? What are their IDs? I don't see any changes. PR:000003328 and PR:000003353 already exist if these are the protein entities you want to add.

@Caroline-99
Copy link
Contributor Author

@Caroline-99 Did you add these proteins to seed.txt? What are their IDs? I don't see any changes. PR:000003328 and PR:000003353 already exist if these are the protein entities you want to add.

Hi, I have added them to the seed.txt, not realising they were already there. Yeah these are the two Ids I want to use in CL.

@aleixpuigb
Copy link
Contributor

If they were in the seed, weren't they available already?

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.

4 participants