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

TS 3.1 fixed #798

Merged
merged 5 commits into from
Sep 20, 2019
Merged

TS 3.1 fixed #798

merged 5 commits into from
Sep 20, 2019

Conversation

crutchcorn
Copy link
Member

@crutchcorn crutchcorn commented Sep 19, 2019

What's Changing and Why

A MINOR semver change to 0.8.3 that fixes the final issues with the 3.1 typings mentioned in #785. The type tests are now EXTREMELY thorough and should have all of the major edgecases and "gotchas" covered

Tasks

  • Add tests
  • Update Documentation
  • Update jimp.d.ts
  • Add SemVer Label

Published PR with canary version: 0.8.4-canary.798.377.0

@crutchcorn
Copy link
Member Author

crutchcorn commented Sep 19, 2019

I had to bump the TS version of Jimp CLI to get it to use the proper types for jimp, which is impatable with jimp/custom (as the <3.1 version of typings of jimp don't work with jimp/custom). I also found an odd edgecase when I did this, so I went ahead and added tests for those and fixed it as well

@patheticcockroach
Copy link

Hi, 0.8.4-canary.798.377.0 works for me 👍

@crutchcorn
Copy link
Member Author

@hipstersmoothie with this PR, the 3.1 typing work I started with the initial 0.8 release should be finally live with really solid tests to prevent the breakage of typings in the future

@hipstersmoothie hipstersmoothie added the patch Increment the patch version when merged label Sep 20, 2019
@hipstersmoothie hipstersmoothie merged commit 8fdc360 into jimp-dev:master Sep 20, 2019
@hipstersmoothie
Copy link
Collaborator

hipstersmoothie commented Sep 20, 2019

🍾 🍾 🍾 🍾 🍾

hooray

@hipstersmoothie
Copy link
Collaborator

🚀 PR was released in v0.8.4 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants