Skip to content
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

Please remove the post install script #757

Closed
pronebird opened this issue Jan 24, 2020 · 9 comments
Closed

Please remove the post install script #757

pronebird opened this issue Jan 24, 2020 · 9 comments

Comments

@pronebird
Copy link

Hi,

It's absolutely terrific that you're looking for a job, but please do not do this via postinstall script. There are job boards, meetups where you can connect with others, LinkedIn, and so forth.

>> node -e "try{require('./postinstall')}catch(e){}"

Cheers,
Andrej

@veeral-patel
Copy link

veeral-patel commented Jan 24, 2020

@pronebird Hey! Looking at other issues in the repository, the maintainer has said that he won't remove the postinstall message, to spite people who want to remove it and maybe for financial reasons as well -- he's mentioned he'll be going to prison in a couple days.

@ljmerza
Copy link

ljmerza commented Jan 25, 2020

where did you see this? If that's true I hope someone forks this project and takes it from him

@veeral-patel
Copy link

#747 (comment)

#548 (comment)

@ljmerza

@veeral-patel
Copy link

@pronebird agree with your original comment. I don't mind the postinstall message, considering all the work he's put into this library for free, but I worry that continuously saying he wants a good job undermines him. I'd expect he be in high demand considering he's authoring a standard library for Javascript, no small undertaking, and I hope this doesn't hurt him.

@ljmerza
Copy link

ljmerza commented Jan 26, 2020

well considering he was committing on a almost daily basis then went silent 12 days ago i would imagine this project just died. here's to hoping babel or someone else takes this over. he sounded like a real winner

@Kein
Copy link

Kein commented Jan 28, 2020

#747 (comment)
#548 (comment)
@ljmerza

Since he is from Russia - I dont think he was joking at all.

@slowcheetah
Copy link
Contributor

@Kein this is not a joke.
@zloirock will be unavailable for some time.
Here he clearly expressed his opinion on this issue.
Closed.

@Daniel15
Copy link

Daniel15 commented Jan 29, 2020

well considering he was committing on a almost daily basis then went silent 12 days ago i would imagine this project just died. here's to hoping babel or someone else takes this over. he sounded like a real winner

@ljmerza just because a project hasn't received commits in two weeks doesn't mean it's dead. People have commitments other than open source work that doesn't make them any money. Also, there is such a thing as a project being 'complete'. Some projects go years between major updates, and that's fine. In fact, often I much prefer libraries that are stable and move slowly. If you really feel some changes to this library are needed, then I'm sure the maintainers would appreciate your pull requests.

@ljmerza
Copy link

ljmerza commented Jan 29, 2020

Yes, prison commitments apparently.

retronav added a commit to retronav/core-js that referenced this issue Apr 12, 2020
When installing multiple packages that depend on core-js, the postinstall message will pop up mutiple times which will add unnecessary verbosity to the terminal log. I know zloirock#767, zloirock#548, zloirock#797, zloirock#757, zloirock#781 ,zloirock#729, zloirock#708, refer to this problem and since @zloirock refuses to remove this, lets limit it to one time so that verbosity gets truncated to the least.
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

No branches or pull requests

6 participants