Closed mmomtchev closed 1 year ago
The new versions of string-width
, wrap-ansi
and strip-ansi
are not only esm, they are esm-only. So I think this PR will break the cjs build of cliui.
There are two recent PR taking different approaches to the problem: #139 #140
If I missed something cunning in this PR that means it does work for both the cjs and esm builds of cliui, let me know!
@shadowspawn this was precisely the point of this PR - use the ESM versions since these were the only ones available. It didn't break the CJS build, because the transpiler took care of it. But nevermind if you have another PR.
Thanks for info. I'll take another look.
Nice, the rollup configuration does work! This is a better direction than I went in #140 where I used the cjs libraries for the esm build to avoid breaking the cjs build.
There are a number of fixes required to the PR. The biggest one is that test:esm
was not being included in npm run test
and is not functional. I tried adding the import for chai to at least get should
functional (as follows), and multiple tests still broken. The early failures are because the wrapping is now working, but there are a couple of later ones too that I have not looked into.
import chai from 'chai'
chai.should()
Are you interested in picking this up again @mmomtchev ? No worries if you are no longer interested, in which case I'll do some combo of this and my attempt and add you as co-author. (Thanks!)
@shadowspawn Don't you have another overlapping PR?
Yes, but your PR was first so I'll help you with yours if you like.
@shadowspawn Maybe I should have started by introducing myself. I have been unemployed for 3 years and I am living with $4 per day - every day I have to make a meal with $2.5. I don't do anything but to work on open-source projects, hoping that I will find a job. I have to do this, despite having 20 years of experience, despite having graduated top of my class and despite having worked incredibly hard every day during my whole life. I am in this situation because I refuse to accept that my supervisor posts dicks and tries to make it look like I was suffering from schizophrenia. My supervisors have been doing this for the last 5 years before my unemployment period. They have been doing this in order to cover up a false rape accusation organized with the help of the French police.
Since I have been unemployed, I had to take this same shit in every open-source project I tried to work in. It happened in the @osgeo projects, then it happended in the @openjsf projects. Because simply posting dicks is not enough, they will usually simultaneously do something to impede my work - and in a way that clearly shows the criminal conspiracy.
I would just like to ask you what is your opinion on this?
It sounds like you have had a difficult time.
I don't think it is worth you working on this old PR. I'll rework my PR and your rollup config and do some new PRs. I want to look at the test setup anyway. I will add you as co-author on the PR with the dependency update and rollup changes.
Thanks for the contribution, and sorry it took a while to rediscover it.
Tell me @shadowspawn, because I am eating with $2.50 per meal and I wouldn't do what you are doing, tell me, please, what is the situation with you
See #143 for an updated PR following through with the approach of updating the dependencies to ESM and using rollup to transpile for the CJS implementation.
Refs: https://github.com/yargs/yargs/issues/2112
string-width
,wrap-ansi
andstrip-ansi
now have ES6 versionsAlso switch all unit tests that require them to ES6