search

jshttp / accepts

Higher-level content negotiation
MIT License
252 stars 42 forks source link

Adding code to run on Powersystem #21

Closed genisysram closed 3 years ago

genisysram commented 3 years ago

Hi Here is my contribution to your code, its working good on powersystems.

Thanks for the code, its working good.

What do these changes do?

Added Architecture "ppc64le"

Are there changes in behavior for the user?

No

dougwilson commented 3 years ago

Thank you for your contribution, but it looks like this does not pass CI, but it is because some of the Node.js versions do not run on ppc64le. Even then, this is a pure-JS module which does not use any OS APIs in Node.js, so there isn't really a need to test on various operation systems and architectures within this project (especially since adding more to the matrix in Travis CI is no longer free).

genisysram commented 3 years ago

I Raised the PR since its getting passing on Latest versions of Node.js