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

Move the CI to use the refactored schematic version #340

Closed
adamjtaylor opened this issue Jan 23, 2024 · 2 comments · Fixed by #351
Closed

Move the CI to use the refactored schematic version #340

adamjtaylor opened this issue Jan 23, 2024 · 2 comments · Fixed by #351
Assignees

Comments

@adamjtaylor
Copy link
Contributor

This should speed up conversion to JSON-LD which is currently our bottleneck.

@adamjtaylor
Copy link
Contributor Author

@aclayton555 @elv-sb per Amy's message here

https://sagebionetworks.slack.com/archives/C01ANC02U59/p1707930913252699

We need to make sure our prod and staging JSON-LD is generated from the latest schematic (at least 24.1.1) before the DCA update tomorrow.

I fear this totally slipped past us. I am traveling without laptop right now but should be able to get us set up with updates and releases by the morning PT tomorrow.

@adamjtaylor
Copy link
Contributor Author

adamjtaylor commented Feb 14, 2024

Not sure why this has been sitting as Done in the project board but left open. Moving back into ready for development.

Not critical for the 15 Feb DCA release but we should relatively promptly update our workflow to uses either a fixed or latest pip released schematicpy, rather than using the poetry install from the GitHub repository.

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 a pull request may close this issue.

1 participant