Skip to content

Update graphql-upload to the latest version 🚀 #6121

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

Merged
2 commits merged into from
Oct 10, 2019

Conversation

greenkeeper[bot]
Copy link
Contributor

@greenkeeper greenkeeper bot commented Oct 10, 2019

The dependency graphql-upload was updated from 8.0.7 to 8.1.0.

This version is not covered by your current version range.

If you don’t accept this pull request, your project will work just like it did before. However, you might be missing out on a bunch of new features, fixes and/or performance improvements from the dependency update.


Publisher: jaydenseric
License: MIT

Release Notes for Version 8.1.0

Minor

  • processRequest now throws an appropriate error when a multipart field value exceeds the configured size limit, fixing #159.
  • When the file size limit is exceeded, mention how many bytes the limit is in the stream error message.
  • Added a new processRequest option to the graphqlUploadExpress and graphqlUploadKoa middleware, for improved testing without mocks or spies which are difficult to achieve with ESM.

Patch

  • Updated dependencies.
  • Due to updated dependencies: Lint fixes, removed redundant eslint-disable-next-line comments, and regenerated the readme.
  • Documented koa-graphql as known to be compatible, via #156.
  • Fixed a readme typo, via #161.
  • Use GitHub Actions instead of Travis for CI.
  • Removed package-lock.json from .gitignore and .prettierignore, as it’s disabled in .npmrc anyway.
  • New file structure.
  • Explicitly defined main exports (instead of using export * from) to prevent accidental public exposure of internal APIs.
  • Moved JSDoc typedefs into the index main entry file, alphabetically sorted.
  • Nicer Browserslist query syntax.
  • Replaced the isObject helper with a smarter and tested isEnumerableObject.
  • Removed the isString helper.
  • Enforced 100% code coverage for tests, and improved processRequest internals and tests (including a new test using vanilla Node.js HTTP), fixing #130 via #162.
  • Removed a workaround from the startServer test helper.
  • Added a new ProcessRequestFunction JSDoc type, and applied it to processRequest.
  • Renamed the UploadOptions JSDoc type to ProcessRequestOptions.
  • Misc. documentation improvements.
Commits

The new version differs by 38 commits.

  • 2223e7a Version 8.1.0.
  • 9de7bfc Update dev dependencies.
  • 0004894 Merge pull request #162 from jaydenseric/coverage-to-100
  • 95a5a3d Better aborted request Koa app error tests.
  • edf7fa4 Remove unnecessary initial value.
  • aed243c Better handling of events related to aborted requests.
  • 3bca7fe Add PR and issue links to the changelog entry.
  • ab8a515 Tweak a test name and update the changelog.
  • 85e8435 Enforce 100% code coverage.
  • 3ffacba Ignore two lines for code coverage
  • 9059bde Test processRequest with Node HTTP
  • 56cf53e fix inadvertant only in test
  • a341dc1 Fix #130 - Error in test logs
  • 46c9513 Remove impossible conditions
  • e98b7c1 Update dev dependencies.

There are 38 commits in total.

See the full diff


FAQ and help

There is a collection of frequently asked questions. If those don’t help, you can always ask the humans behind Greenkeeper.


Your Greenkeeper bot 🌴

@ghost ghost added the Merge On Green label Oct 10, 2019
@ghost ghost merged commit aef972b into master Oct 10, 2019
@codecov
Copy link

codecov bot commented Oct 10, 2019

Codecov Report

Merging #6121 into master will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6121      +/-   ##
==========================================
+ Coverage   93.73%   93.76%   +0.03%     
==========================================
  Files         165      165              
  Lines       11199    11199              
==========================================
+ Hits        10497    10501       +4     
+ Misses        702      698       -4
Impacted Files Coverage Δ
src/RestWrite.js 93.72% <0%> (+0.16%) ⬆️
src/Adapters/Storage/Mongo/MongoStorageAdapter.js 92.97% <0%> (+0.7%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8aa0687...7e523ed. Read the comment docs.

@greenkeeper greenkeeper bot deleted the greenkeeper/graphql-upload-8.1.0 branch October 10, 2019 01:09
UnderratedDev pushed a commit to UnderratedDev/parse-server that referenced this pull request Mar 21, 2020
* fix(package): update graphql-upload to version 8.1.0

* chore(package): update lockfile package-lock.json
This pull request was closed.
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.

0 participants