Comments (5)
I'm afraid you can't claim !s
as your own invention. It's a pretty obvious fix by looking at the stack trace.
I'm cool with borrowing ideas from each other too, of course.
from colorette.
Sorry, but I must disagree on behalf of the little programmer out there. You just can't take credit for a !s
.
!s
was the smallest change I had to make to avoid the explicit s === undefined
, so I went with that. I could've reversed the expression just as well and not use !s
at all:
!(!s && (s === "" || s === undefined))
s || (s !== "" && s !== undefined)
s || !(s === "" || s === undefined)
They're all roughly equivalent and produce the same result. I'll tell you what, I'll switch to another one and close here.
from colorette.
You edge case suggestions was very useful, but you didn’t mention !s
fix there. You didn’t create an initial boost, and you didn’t fix it to name yourself as author.
I most of the cases, I am OK with idea copying between projects. But in this case, I think I have the right to do another way and ask to mention me if you copied this performance boost. You may mention me in a way “Initially create in Nano Colors and then fixed by team work of me and Andrey Sitnik”.
from colorette.
Anyway, it was not did by you.
And it was a fix for initial my idea that === ''
compare is slow. Without that changes, this discussion would not be started.
You do not need this fix in Colorette for undefined
argument. You added !s
check only because you saw that it was very effective in Nano Colors.
from colorette.
You lost the whole idea of optimization.
You saw from Nano Colors that s !== "" && s !== undefined
is slow. You copied the fix of this slow part.
Now you are changing !x
to x
instead of do an honor thing and mention an author of original work.
from colorette.
Related Issues (20)
- Use process.stdout.isTTY instead of node:tty HOT 8
- Error [ERR_REQUIRE_ESM]: Must use import to load ES Module HOT 3
- Support for CI environments is lacking HOT 7
- require() of ES modules is not supported HOT 5
- [colorette v1.4] `tty && tty.isatty(1)` breaks in vite HMR mode HOT 4
- Upgrading to 2.0.0 HOT 1
- `createColors` has a required argument in d.ts HOT 1
- [using pino-pretty] issue Cannot read property 'env' of undefined ==> const env = process["default"].env || {} HOT 3
- tty.isatty is not a function HOT 2
- Default exports break down in converted CJS version HOT 4
- Support `--color` and `--no-color` flags? HOT 14
- Add kleur and kleur’s author mention to docs as inspiration HOT 5
- Broken SemVer HOT 14
- Regression in 2.x? Disable colors globally for tests HOT 9
- SvelteKit + colorette HOT 2
- Wrong support detection on Windows HOT 2
- Uncaught (in promise) TypeError: Cannot use 'in' operator to search for 'NO_COLOR' in undefined HOT 2
- TS: `Color` should accept `text` of `any` type HOT 1
- [Feature Request] Modifiers support `link`. HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from colorette.