Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Prepare for ODK 1.5 #1794

Merged
merged 7 commits into from
Mar 4, 2024
Merged

Prepare for ODK 1.5 #1794

merged 7 commits into from
Mar 4, 2024

Conversation

gouttegd
Copy link
Contributor

Similar to FlyBase/drosophila-developmental-ontology#92 for FBdv, this PR prepares the FBbt repository for the upcoming release of ODK 1.5.

In addition to removing the step that installs libbusiness-isbn-perl (not needed anymore as the package is already included in the new ODK), this removes the custom step that installs the FlyBase ROBOT plugin, to let the ODK take care of the installation instead.

The libbusiness-isbn-perl package is now part of the ODK, we don't have
to install it on our side.
Do not manually install the FlyBase ROBOT plugin in fbbt.Makefile. Let
the ODK take care of that.
@gouttegd gouttegd marked this pull request as draft February 29, 2024 15:58
@gouttegd gouttegd self-assigned this Feb 29, 2024
@gouttegd gouttegd marked this pull request as ready for review March 4, 2024 10:35
@gouttegd gouttegd marked this pull request as draft March 4, 2024 10:36
There's no need to download mirrored ontology in the test suite since we
are not refreshing imports, but with IMP=false only they could still get
downloaded. We need MIR=false to ensure they are not.
Commit all files re-generated or added by make update_repo.
@gouttegd gouttegd marked this pull request as ready for review March 4, 2024 12:00
@gouttegd gouttegd requested a review from Clare72 March 4, 2024 12:36
@Clare72
Copy link
Contributor

Clare72 commented Mar 4, 2024

Copy link
Contributor

@Clare72 Clare72 left a comment

Choose a reason for hiding this comment

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

approving this, but we need to keep an eye on imported relationships to make sure we don't lose any inference in future (until INCATools/ontology-development-kit#1024 is fixed)

@gouttegd gouttegd merged commit 3ca6c6b into master Mar 4, 2024
1 check passed
@gouttegd gouttegd deleted the prepare-for-odk15 branch March 4, 2024 17:57
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