-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Replace advertising in npm postinstall message with real advertisement #571
Comments
Perfect idea. But we can go further - advertising in the browser console, loading google adverts to sites which use |
This comment has been minimized.
This comment has been minimized.
@Arcanemagus sorry, your message is offtopic, you could write it in #548. It's a thread about replacing the |
But guys, this package is not a spacial snowflake, there are many open source packages and if everybody starts to put advertisements it will be impossible to work. @zloirock I think you should remove the terminal advertisement messages because it makes you look desperate. Readme already contains enough info for supporters. Mind you, at any time any open source package can be forked and ads can be removed this way. I'm not threatening, just saying. |
@revelt Forking is one thing, maintaining is another ;) |
Closed for preventing spam. |
This package is quite popular, and if you are so desperate to get money and keep the package free and bother literally thousands with 15 lines of messages, then why not go all the way and introduce actual advertisements, i am sure there exists a package for that, that could pull google adverts directly to the command line. Untapped market i say.
The text was updated successfully, but these errors were encountered: