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

Fix edgeHandling types #1080

Merged

Conversation

domdomegg
Copy link
Contributor

@domdomegg domdomegg commented Apr 4, 2022

What's Changing and Why

Fixes the edgeHandling type definitions and documentation around convolution.

Why is number correct?

A future improvement may be to make all functions that expect constants to be typed to only accept those constants. But I think that's beyond the scope of this PR for now.

I have also updated the documentation that preivously had a comment 'make me better' to give a more practical example of how to call the convolution method.

Tasks

  • Add tests: Not done because just updating docs and types (and there are no type tests as far as I can see)
  • Update Documentation
  • Update jimp.d.ts
  • Add SemVer Label: Happy to add in PR, but assumed library maintainers would want to do that centrally to avoid conflicts?
📦 Published PR as canary version: 0.16.2-canary.1080.1288.0

✨ Test out this PR locally via:

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

@hipstersmoothie hipstersmoothie merged commit 6ec3e04 into jimp-dev:master Feb 3, 2023
@domdomegg
Copy link
Contributor Author

Thanks for the review and merge @hipstersmoothie 🙌

@domdomegg domdomegg deleted the domdomegg/fix-edgeHandling-types branch February 3, 2023 20:38
@hipstersmoothie
Copy link
Collaborator

🚀 PR was released in v0.16.13 🚀

@hipstersmoothie hipstersmoothie added the released This issue/pull request has been released. label Feb 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants