Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • G GoCardless
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 11
    • Issues 11
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Extensions
  • GoCardless
  • Merge requests
  • !80

Remove extra newlines from logging

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Rich requested to merge github/fork/jmdh/logging_tidy into master Apr 29, 2020
  • Overview 3
  • Commits 1
  • Pipelines 0
  • Changes 1

Created by: jmdh

Some log processing tools don't handle empty lines well, so remove them. This also improves readability, attaching timestamps to content more closely.

Note that the core logging functionality also introduces additional newlines, which should arguably be removed as well. I've applied the trivial patch to my local installation to fix this there, but I need to look more carefully at callers to see what the impact is. Meanwhile this patch should be harmless.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/jmdh/logging_tidy