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

Add single frame encoder for type-gif #899

Merged

Conversation

jeffbseeking
Copy link
Contributor

@jeffbseeking jeffbseeking commented May 31, 2020

What's Changing and Why

There wasn't a encoder for image/gif so any attempt to create a buffer with mimetype image/gif would fail with the error Unsupported MIME type: image/gif

What else might be affected

Not sure how things hook up with the getBuffer method so could use some guidance here. All the other encoder functions in the other types seemed synchronous.

Tasks

  • Add tests
  • Update Documentation
  • [N/A] Update jimp.d.ts
  • Add SemVer Label
📦 Published PR as canary version: 0.13.0-canary.899.964.0

✨ Test out this PR locally via:

npm install @jimp/cli@0.13.0-canary.899.964.0
npm install @jimp/core@0.13.0-canary.899.964.0
npm install @jimp/custom@0.13.0-canary.899.964.0
npm install jimp@0.13.0-canary.899.964.0
npm install @jimp/plugin-blit@0.13.0-canary.899.964.0
npm install @jimp/plugin-blur@0.13.0-canary.899.964.0
npm install @jimp/plugin-circle@0.13.0-canary.899.964.0
npm install @jimp/plugin-color@0.13.0-canary.899.964.0
npm install @jimp/plugin-contain@0.13.0-canary.899.964.0
npm install @jimp/plugin-cover@0.13.0-canary.899.964.0
npm install @jimp/plugin-crop@0.13.0-canary.899.964.0
npm install @jimp/plugin-displace@0.13.0-canary.899.964.0
npm install @jimp/plugin-dither@0.13.0-canary.899.964.0
npm install @jimp/plugin-fisheye@0.13.0-canary.899.964.0
npm install @jimp/plugin-flip@0.13.0-canary.899.964.0
npm install @jimp/plugin-gaussian@0.13.0-canary.899.964.0
npm install @jimp/plugin-invert@0.13.0-canary.899.964.0
npm install @jimp/plugin-mask@0.13.0-canary.899.964.0
npm install @jimp/plugin-normalize@0.13.0-canary.899.964.0
npm install @jimp/plugin-print@0.13.0-canary.899.964.0
npm install @jimp/plugin-resize@0.13.0-canary.899.964.0
npm install @jimp/plugin-rotate@0.13.0-canary.899.964.0
npm install @jimp/plugin-scale@0.13.0-canary.899.964.0
npm install @jimp/plugin-shadow@0.13.0-canary.899.964.0
npm install @jimp/plugin-threshold@0.13.0-canary.899.964.0
npm install @jimp/plugins@0.13.0-canary.899.964.0
npm install @jimp/test-utils@0.13.0-canary.899.964.0
npm install @jimp/bmp@0.13.0-canary.899.964.0
npm install @jimp/gif@0.13.0-canary.899.964.0
npm install @jimp/jpeg@0.13.0-canary.899.964.0
npm install @jimp/png@0.13.0-canary.899.964.0
npm install @jimp/tiff@0.13.0-canary.899.964.0
npm install @jimp/types@0.13.0-canary.899.964.0
npm install @jimp/utils@0.13.0-canary.899.964.0
# or 
yarn add @jimp/cli@0.13.0-canary.899.964.0
yarn add @jimp/core@0.13.0-canary.899.964.0
yarn add @jimp/custom@0.13.0-canary.899.964.0
yarn add jimp@0.13.0-canary.899.964.0
yarn add @jimp/plugin-blit@0.13.0-canary.899.964.0
yarn add @jimp/plugin-blur@0.13.0-canary.899.964.0
yarn add @jimp/plugin-circle@0.13.0-canary.899.964.0
yarn add @jimp/plugin-color@0.13.0-canary.899.964.0
yarn add @jimp/plugin-contain@0.13.0-canary.899.964.0
yarn add @jimp/plugin-cover@0.13.0-canary.899.964.0
yarn add @jimp/plugin-crop@0.13.0-canary.899.964.0
yarn add @jimp/plugin-displace@0.13.0-canary.899.964.0
yarn add @jimp/plugin-dither@0.13.0-canary.899.964.0
yarn add @jimp/plugin-fisheye@0.13.0-canary.899.964.0
yarn add @jimp/plugin-flip@0.13.0-canary.899.964.0
yarn add @jimp/plugin-gaussian@0.13.0-canary.899.964.0
yarn add @jimp/plugin-invert@0.13.0-canary.899.964.0
yarn add @jimp/plugin-mask@0.13.0-canary.899.964.0
yarn add @jimp/plugin-normalize@0.13.0-canary.899.964.0
yarn add @jimp/plugin-print@0.13.0-canary.899.964.0
yarn add @jimp/plugin-resize@0.13.0-canary.899.964.0
yarn add @jimp/plugin-rotate@0.13.0-canary.899.964.0
yarn add @jimp/plugin-scale@0.13.0-canary.899.964.0
yarn add @jimp/plugin-shadow@0.13.0-canary.899.964.0
yarn add @jimp/plugin-threshold@0.13.0-canary.899.964.0
yarn add @jimp/plugins@0.13.0-canary.899.964.0
yarn add @jimp/test-utils@0.13.0-canary.899.964.0
yarn add @jimp/bmp@0.13.0-canary.899.964.0
yarn add @jimp/gif@0.13.0-canary.899.964.0
yarn add @jimp/jpeg@0.13.0-canary.899.964.0
yarn add @jimp/png@0.13.0-canary.899.964.0
yarn add @jimp/tiff@0.13.0-canary.899.964.0
yarn add @jimp/types@0.13.0-canary.899.964.0
yarn add @jimp/utils@0.13.0-canary.899.964.0

@hipstersmoothie
Copy link
Collaborator

This is looking good. get the lint passing and I'll merge 🚀

@jeffbseeking
Copy link
Contributor Author

Fixed linting @hipstersmoothie

@hipstersmoothie hipstersmoothie added the minor Increment the minor version when merged label Jun 5, 2020
@hipstersmoothie
Copy link
Collaborator

The version for this project is automated through GitHub labels using https://github.com/intuit/auto. Thanks for thinking about it though!

@hipstersmoothie hipstersmoothie merged commit fcc5b23 into jimp-dev:master Jun 5, 2020
@jeffbseeking jeffbseeking deleted the add-single-frame-gif-encoder branch June 5, 2020 05:58
@hipstersmoothie
Copy link
Collaborator

🚀 PR was released in v0.13.0 🚀

@hipstersmoothie hipstersmoothie added the released This issue/pull request has been released. label Jun 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor Increment the minor version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants