Comments (15)
Hi, thanks for the input. Can you explain me why I should use the formatter? Can you post a screenshot?
from nest-winston.
It was this format that lead to my example in #44, but here it is again (note the colours on the console are the same as what the Nest Logger uses)
info: 9/28/2019, 5:24:14 PM [InstanceLoader] TypeOrmCoreModule dependencies initialized
info: 9/28/2019, 5:24:14 PM [RoutesResolver] AppController {/}:
from nest-winston.
The payk repo is actually the one that this author stole 99% of the code from... This repo only had an export module before my repo. Just look at the PRs.
from nest-winston.
Your created the repo just 26 days ago. What you are saying is a no-sense
from nest-winston.
Check the pull requests, all the functionality in this repo was literally written by me
from nest-winston.
@danshapir yes, that functionality, not the entire project.... i did not stolen anything. You cloned my repo and do you things, which is perfectly right.... this is open source code. So you should really stop saying "stole".
from nest-winston.
I think the problem is that the fork link between this repo and https://github.com/pay-k/nestjs-winston is broken (assuming one was present in the first place) so when someone new comes alone it's easy to get confused. There's also no point having multiple copies of the same module in the npm registry IMHO.
I also don't think it's fair to claim @gremo stole anything. Firstly the license is MIT, so noone can actually steal anything. Secondly the merge commits show that the code was donated/merged in willingly #36
from nest-winston.
@gremo I wish this was true. The last PR I closed and deleted, but still you went on and took my code...
And taking the code from this issue will do the same again. Copy paste...
from nest-winston.
@danshapir Again, I did not stolen anything. You contributed to this project as well as other people of this community. Thanks god, we still write open source code and the verb "stole" is completely inappropriate.
I was happy with your contribution but you were not able to respect the coding standard of this project and for this reason I authored that commit by myself.
Feel free to contribute again but without making a real mess with your commits. There is a file called tslint.json
for this purpose. If you don't want to respect the rules, or you want to create a commit with your phone instead of using VSCode, then we can stop talking about this right now.
@kierans Can you make a PR with the code coming from @danshapir ?
from nest-winston.
@kierans there are other stolen PRs.
Just like the comment above me asking you to make a PR with code coming from my repo
Common, I'm actually keeping my repo updated all the time, adding functionality and features.
Just use it instead on copying stuff from it
from nest-winston.
@danshapir I understand your point that you may not have agreed to have code that you've written added to this repository. However both your repos are licensed under the MIT license. Therefore @gremo cannot, and has not "stolen" anything, so it's inappropriate for you to say so. If you wish your intellectual property to remain your own, you need to have a less permissive license on your code. Looking at the commit history it's also easy to see your name on commits so attribution is present.
from nest-winston.
Personally I think everybody suffers from this type of split. What let me to this repo, is that @gremo has the nestjs-winston
npm package and I was trying to figure out what the difference between it and @payk/nestjs-winston
was.
What would be better is if somehow all code could be merged into one package and published.
from nest-winston.
This conversation is ridiculous. This is a small library with a few stars (30 or so) and downloads. There no reason to complain about the intellectual property of your work.
@danshapir there are no "other PRs", the only one I changed was #36 because you were NOT able to simply lint your code, even after i told you to do so. You are not able to collaborate with other people and do something for the community, for free.
@kierans I will work to port the new formatter from @danshapir repository.
from nest-winston.
@kierans I will work to port the new formatter from @danshapir repository.
Thanks. I don't have the time currently.
from nest-winston.
@kierans what do you think? Last 4 lines are the custom nest-like formatter I'm writing:
Another format (I like):
from nest-winston.
Related Issues (20)
- nest-winston v1.6.3 has runtime error ERR_REQUIRE_ESM HOT 11
- Extend the log methods HOT 4
- Log unhandled exceptions in readable form HOT 3
- WinstonLogger suggestions HOT 2
- Write after end node error with winston and nest-winston logger HOT 4
- TypeError: Cannot destructure property 'message' of 'message' as it is null
- Logging NestJS request details with every log? HOT 4
- "Unknown logger level" error when used for bootstraping HOT 3
- Nest can't resolve dependencies HOT 8
- Default context per service (configurable injection scope) HOT 3
- Passing winston logger instance to forRoot doesn't work HOT 2
- [Bug]: Creating multiple winston modules using child winston loggers use the last created winston module when using the nestjs Logger constructor HOT 2
- Adding "level" argument to log method HOT 2
- NestLike ConsoleFormat always contains stringifiedMeta and essentially logs message twice HOT 7
- LoggerService interface changed with Nest v8 HOT 2
- Is there a way to create a child logger? HOT 2
- How can I pass json attributes into the logger? HOT 3
- Support for @nestjs/common 10.0.0 in Peer Dependencies HOT 2
- [Question]: Should documentation be updated? HOT 8
- [Feature]: Support string-interpolation 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 nest-winston.