angularjs-logDecorator

angularjs-logDecorator copied to clipboard

Potential license confusion

1

The `LICENSE` file denotes an MIT license, but [the `README.md`'s "License" heading](https://github.com/ThomasBurleson/angularjs-logDecorator/blob/master/README.md#license) shows "other things" (a custom license, it seems). This implies a dual-licensing, to me. Please clarify this aspect;...

nmschulte

Make AMD module 'angular-logX' the main AMD module

3

After merging this PR, we'd have the situation: 1. If you need a decorated logger, just require 'mindspace/logger/ExternalLogger' 2. If you need decorated loggers with AngularJS, require 'angular-logX' and add...

k7sleeper

Looks like copy and paste strikes again. The line in the Demo #5 section code on the README.md has a mistake, the actual code is correct in the repository however...

Zabeard

Remove dependency on Require?

7

I am contemplating using this or something like it with a project here, but we're not using Require as part of the project. Have you thought about removing that (and...

tommck

Readme Images Repository

1


ThomasBurleson

Hey, first of all: Great Decorator - Example, thanks! I often use $log.info with objects and often with multiple params, to show all properties, but with your decorator these are...

negue