-
-
Notifications
You must be signed in to change notification settings - Fork 234
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
chore: Bump @stoplight/json-ref-resolver from 2.2.0 to 2.3.0 #664
Conversation
@P0lip Could you please take a look a this when you've got some spare time? |
src/__tests__/spectral.test.ts
Outdated
({ | ||
result: {}, | ||
refMap: {}, | ||
errors: [], |
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.
Let's avoid the type trickery here.
Add the following import
import { DepGraph } from 'dependency-graph';
and then you should be able to include graph
errors: [],
graph: new DepGraph(),
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.
@P0lip My bad. It's now fixed.
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.
LGTM 👍
Will merge as soon as the CI checks are green.
@P0lip Shouldn't we also explicitly bump the reference in |
Ah, yeah, we should do that. |
@nulltoken could you bump it? |
Bumps [@stoplight/json-ref-resolver](https://github.com/stoplightio/json-ref-resolver) from 2.2.0 to 2.3.0. - [Release notes](https://github.com/stoplightio/json-ref-resolver/releases) - [Commits](stoplightio/json-ref-resolver@v2.2.0...v2.3.0) Signed-off-by: dependabot-preview[bot] <[email protected]>
Bumped! |
Needed by #635
Checklist
Does this PR introduce a breaking change?
wrt "hacking the tests", this isn't the ideal way to do this, but as the they do not depend
on the newly introduced graph member, that should be enough.
Relevant context: