-
Notifications
You must be signed in to change notification settings - Fork 29
chore: update deps and require npm@7 #253
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
Conversation
"main": "src/index.js", | ||
"scripts": { | ||
"lint": "eslint .", | ||
"pretest": "npm run lint && npm run prettier", |
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.
I've started removing these wherever I see them because our ESLint config already runs Prettier, so this is always going to run Prettier on an already prettified codebase.
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!
🧰 What's being changed?
@readme/httpsnippet
to the latest release and moves it from being a dep to a peerDep.@readme/oas-to-har
to the latest release to pull in some fixes w/r/tformat: json
request bodies. fix(oas-to-har): clean up formatting of raw json bodies api-explorer#1197oas
to the latest version across bothapi
andhttpsnippet-client-api
.getAuth
method intoOas
(feat: addition of a newgetAuth
method on the OAS class oas#376) but we've been unable to take advantage of it because we have some semver clases in ReadMe and it's preferring 10.0.0 over 10.2.0 becausehttpsnippet-client-api
had 10.0.0 listed. Incredibly frustrating, but this should let us fully upgradeoas
everywhere.oas
to being a peerDep inhttpsnippet-client-api
. Sincehttpsnippet-client-api
is a bit more volatile thanapi
, it makes sense to move it to being a peerDep so we can have a bit more leeway when installing it in other other packages and frees us up from having to always do version bumps onoas
.