jimp-dev / jimp-dev/gifwrap

gifwrap has circular dependency in it. I have a patch to remove that.

Open
#48 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
77
Forks
18
PR merge metrics
No merged PRs in 30d

Description

Hi, I was building jimp and saw that the browser build was giving error that gifwrap has circular dependency and so i took some time to remove that. If maintainers think it is good idea then it can be merged.

My reasoning here was utilities are used by all but they are not dependent on any particular service or functionality or feature.

My PR is here: https://github.com/jimp-dev/gifwrap/pull/47

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.