Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added editing policy #51

Merged

Conversation

hauschke
Copy link
Member

VIVO-Ontology GitHub issue:

What does this pull request do?

Adds an editing policy to the readme to only allow ontology edits with Protégé.

Interested parties

@vivo-ontologies/team-members

Only Protégé edits, no manual editing.
@hauschke
Copy link
Member Author

@hauschke
Copy link
Member Author

2nd test by @StroemPhi: hauschke#2

It is important to do a proper reserialization. The PR for this needs to be carefully checked.

@hauschke
Copy link
Member Author

Decision in https://wiki.lyrasis.org/display/VIVO/2024-10-02+Ontology+Interest+Group+Call:

  • Create recommendations for users:
    -- Use Protege for big modifications and re-serialize the result before creating a PR with robot tool to minimize difference to review.
    -- For small modifications when modification can be done in text editor no re-serialization is needed.

Copy link
Contributor

@chenejac chenejac left a comment

Choose a reason for hiding this comment

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

@hauschke looks very good. Please, consider my tiny suggestion before this PR is merged.

README.md Outdated Show resolved Hide resolved
Co-authored-by: Dragan Ivanovic <chenejac@uns.ac.rs>
@brianjlowe brianjlowe merged commit b1c8b2b into vivo-ontologies:master Oct 23, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants