l2met-compatible-logging #6

Merged
merged 3 commits into from Mar 5, 2013

Conversation

Projects
None yet
3 participants
@jkakar
Contributor

jkakar commented Mar 5, 2013

  • Vault::Log methods emit L2met-compatible output. When an
    APP_NAME environment variable is present it will be prepended to
    measurement names.
  • Tests for the Vault::Log module are now in place.
  • Docstrings use YARD syntax now.

jkakar added some commits Mar 2, 2013

- Fix breaking test.
- Initial work on refactoring logging logic.
- Add tests for Vault::Log module.
- Update logging functionality to be compatible with L2met.
@csquared

This comment has been minimized.

Show comment
Hide comment
@csquared

csquared Mar 5, 2013

Contributor

👍

Contributor

csquared commented Mar 5, 2013

👍

@mattmanning

This comment has been minimized.

Show comment
Hide comment
@mattmanning

mattmanning Mar 5, 2013

Contributor

+1

Contributor

mattmanning commented Mar 5, 2013

+1

mattmanning added a commit that referenced this pull request Mar 5, 2013

Merge pull request #6 from heroku/l2met-compatible-logging
l2met-compatible-logging

- `Vault::Log` methods emit L2met-compatible output.  When an
  `APP_NAME` environment variable is present it will be prepended to
  measurement names.

- Tests for the `Vault::Log` module are now in place.

- Docstrings use YARD syntax now.

@mattmanning mattmanning merged commit ce2ec08 into master Mar 5, 2013

@mattmanning mattmanning deleted the l2met-compatible-logging branch Mar 5, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment