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

Enable usage testing of the preApplication schema #290

Merged
merged 4 commits into from
Jan 16, 2025

Conversation

pixeltrix
Copy link
Contributor

This PR does the following:

  1. Rebuilds commit 5ddbba6 using the correct dependencies
  2. Fixes some lint warnings
  3. Ensures build-schema fails when a subprocess fails
  4. Extracts Entity to a separate file to avoid duplicate definition
  5. Ignores the build directory which contained tsconfig.buildinfo
  6. Adds the preApplication schema to the usage tests

Copy link
Member

@jessicamcinchak jessicamcinchak left a comment

Choose a reason for hiding this comment

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

Thanks for this !

@@ -0,0 +1,16 @@
import {URL} from './utils';
Copy link
Member

Choose a reason for hiding this comment

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

Also solved the duplicate Entity definition error via #289 - sorry for conflicts here!

name: 'PreApplication',
schema: preApplicationSchema,
examples: getJSONExamples<PreApplication>('preApplication'),
},
Copy link
Member

Choose a reason for hiding this comment

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

Much appreciated - fixes #243

@@ -1,5 +1,8 @@
#!/bin/bash

set -e
set -o pipefail
Copy link
Member

Choose a reason for hiding this comment

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

👍 much nicer

@pixeltrix
Copy link
Contributor Author

@jessicamcinchak rebase to fix the merged conflicts. There was a rollback to version 6.0 of pnpm-lock.yaml in #291 which I think was unintentional so reverting that to version 9.0 is included.

@jessicamcinchak
Copy link
Member

Thanks again @pixeltrix - going to merge this and then rebase and merge #287, then main will be fully representative of next queued release and we can all take a final look 👍

@jessicamcinchak jessicamcinchak merged commit 3938c4a into theopensystemslab:main Jan 16, 2025
1 check passed
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