-
Notifications
You must be signed in to change notification settings - Fork 57
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
Add asdf-unit-schemas as dependency #1210
Add asdf-unit-schemas as dependency #1210
Conversation
0154bc4
to
2799ffe
Compare
2799ffe
to
139ef00
Compare
Any clue why the |
Thats because they aren't being run anymore. I replaced them with a test matrix which runs the standard, transform, and unit tests. It only shows up because its marked as required currently. |
asdf-standard: | ||
name: Run asdf-standard tests | ||
asdf-schemas: | ||
name: ${{ matrix.package_name }}@${{ matrix.ref }} unit tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This changes the name
Should this be merged then the requirements changed or the opposite? |
We cannot add the new CI as a requirement until this PR is merged. So we merge and then update the requirements. |
Shortly, we will drop
units
fromasdf-standard
in favor of theasdf-unit-schemas
. For backwards compatibility reasons, we then need to addasdf-unit-schemas
as a dependency toasdf
.