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

Feature/procedural cleanup #238

Merged
merged 10 commits into from
Apr 3, 2023
Merged

Feature/procedural cleanup #238

merged 10 commits into from
Apr 3, 2023

Conversation

denis-yuen
Copy link
Member

@denis-yuen denis-yuen commented Feb 16, 2023

WIP with documentation update and some simple community recommended tickets

Addresses

@denis-yuen denis-yuen self-assigned this Feb 16, 2023
@denis-yuen
Copy link
Member Author

Note to self: I also tried to fix the 2.0.1 documentation builds which seem troubled. It looks like documentation builds on tags as opposed to branches seems broken.

Looks possibly like ga4gh/gh-openapi-docs#43 but the fix is unreleased in the develop branch if I understand correctly. In the meantime, I created a branch https://github.com/ga4gh/tool-registry-service-schemas/tree/release-2.0.1 so that documentation for that release shows up in the documentation table of contents

@denis-yuen
Copy link
Member Author

Note for self: need to put back swagger.json link in addition to openapi link to get both since we go "back in time" when generating a TOC

@denis-yuen denis-yuen marked this pull request as ready for review February 22, 2023 15:15
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@denis-yuen denis-yuen requested a review from coverbeck February 27, 2023 22:41
@denis-yuen denis-yuen merged commit 4204d76 into develop Apr 3, 2023
@denis-yuen denis-yuen deleted the feature/procedural_cleanup branch April 3, 2023 14:23
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