8000 Bump color-rgba 2.1.1 - module color-normalize by archmoj · Pull Request #4 · colorjs/color-normalize · GitHub
[go: up one dir, main page]

Skip to content
8000

Bump color-rgba 2.1.1 - module color-normalize #4

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
merged 3 commits into from
Jun 25, 2019
Merged

Bump color-rgba 2.1.1 - module color-normalize #4

merged 3 commits into from
Jun 25, 2019

Conversation

archmoj
Copy link
Contributor
@archmoj archmoj commented May 25, 2019

Include fix for colorjs/color-parse#1
@dy
cc: @etpinard

@archmoj
Copy link
Contributor Author
archmoj commented May 26, 2019

It seems that one test is not compatible with current "stable" node version 12.3.1.
The previous successful run on a "stable" version was performed on node version 10.5.0: https://travis-ci.org/colorjs/color-normalize/jobs/395637825.

@etpinard
Copy link

@archmoj do you have an idea why tests are failing in node 12?

@archmoj
Copy link
8000 Contributor Author
archmoj commented May 28, 2019

The issue was with node 12 & not with color-rgba 2.1.1 as reported here: #3 (tests are failed even on the master branch when using node 12)!

The test is fixed to work with node 12.
@dy would you mind having the second look at this PR?
Thanks.

@archmoj
Copy link
Contributor Author
archmoj commented Jun 5, 2019

@dy could you possibly release a patch from this?
Many thanks.

@zeke
Copy link
Member
zeke commented Jun 13, 2019

Looks good to me. I'd merge it, but I'm not an owner of the npm package.

@archmoj
Copy link
Contributor Author
archmoj commented Jun 14, 2019

Thanks @zeke for the review.
Great news!
Then we could wait for @dy for the npm package.

@archmoj
Copy link
Contributor Author
archmoj commented Jun 24, 2019

@dy any chance we have a patch release this week?
Thanks.

@dy dy merged commit b74673a into colorjs:master Jun 25, 2019
@@ -1,14 +1,21 @@
const assert = require('assert')
const rgba = require('./')

function assert_deepEqual(a, b) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@archmoj what was the cause for this method?

@dy
Copy link
Member
dy commented Jun 25, 2019

Sorry for the delay. Done.

@archmoj archmoj deleted the rgb-has-no-alpha branch June 25, 2019 19:29
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.

4 participants
0