Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: hashicorp/copywrite
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.16.6
Choose a base ref
...
head repository: hashicorp/copywrite
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v0.17.0
Choose a head ref
  • 2 commits
  • 4 files changed
  • 2 contributors

Commits on Nov 13, 2023

  1. fix(logging): respect log levels for skipped files (#102) 

    Previously, the `copywrite headers` command would output a lot of `[DEBUG] skipping file...` statements, esp. when large folders were excluded (looking at you, `NODE_MODULES`). This made for a poor experience, and was a bug introduced when I made #80 - as in an attempt to fix an issue with GHA log group directives and addlicense sending to stdout and stderr, respectively, I introduced a bug that ignored the `COPYWRITE_LOG_LEVEL` env var.

This PR fixes that error while ensuring the GHA log grouping still works by redirecting the `cliLogger` we use throughout the headers cmd (and others) to stdout by default. This doesn't appear to be a large shift or change for other commands, but does make a big difference for the headers command.
    @CalebAlbers
    CalebAlbers authored Nov 13, 2023
    Configuration menu
    Copy the full SHA
    9f8cdf2 View commit details
    Browse the repository at this point in the history

  2. Add support for .lr and .lr.go files (#85) 

    Signed-off-by: Christian Zunker <christian.zunker@gmail.com>
    @czunker
    czunker authored Nov 13, 2023
    Configuration menu
    Copy the full SHA
    9103d86 View commit details
    Browse the repository at this point in the history
Loading