Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FR]: 'configure' should support logging, indicate progress #428

Open
jbedard opened this issue Apr 25, 2023 · 0 comments
Open

[FR]: 'configure' should support logging, indicate progress #428

jbedard opened this issue Apr 25, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@jbedard
Copy link
Member

jbedard commented Apr 25, 2023

What is the current behavior?

Gazelle is configured to log level WARN and can't be changed:

BazelLog.Level = logrus.WarnLevel

Describe the feature

Ability to set a log level or verbose flag for aspect configure.

@jbedard jbedard added the enhancement New feature or request label Apr 25, 2023
@jbedard jbedard self-assigned this Apr 25, 2023
@github-actions github-actions bot added the untriaged Requires traige label Apr 25, 2023
@jbedard jbedard removed the untriaged Requires traige label Jun 12, 2023
gregmagolan pushed a commit that referenced this issue Sep 22, 2023
Moving the gazelle logging util methods to a common util for all of the
cli (cli/core/pkg/logger/logger.go). For now this util is the basic go
`log` package, if we continue using this util for logging then we can
replace the underlying log library with anything in the future.

For now there are 2 undocumented env vars to configure logging level +
output file.
The log file + level is auto configured when used within `bazel test`
which has been used by the gazelle tests for quite some time.

Ref:
#428
#480
#290
#26
GitOrigin-RevId: dd5b7949117ec4298671945d7d0df1ba288b9d8b
jbedard added a commit that referenced this issue Oct 4, 2023
Moving the gazelle logging util methods to a common util for all of the
cli (cli/core/pkg/logger/logger.go). For now this util is the basic go
`log` package, if we continue using this util for logging then we can
replace the underlying log library with anything in the future.

For now there are 2 undocumented env vars to configure logging level +
output file.
The log file + level is auto configured when used within `bazel test`
which has been used by the gazelle tests for quite some time.

Ref:
#428
#480
#290
#26
GitOrigin-RevId: dd5b7949117ec4298671945d7d0df1ba288b9d8b
jbedard added a commit that referenced this issue Oct 4, 2023
Moving the gazelle logging util methods to a common util for all of the
cli (cli/core/pkg/logger/logger.go). For now this util is the basic go
`log` package, if we continue using this util for logging then we can
replace the underlying log library with anything in the future.

For now there are 2 undocumented env vars to configure logging level +
output file.
The log file + level is auto configured when used within `bazel test`
which has been used by the gazelle tests for quite some time.

Ref:
#428
#480
#290
#26
GitOrigin-RevId: dd5b7949117ec4298671945d7d0df1ba288b9d8b
jbedard added a commit that referenced this issue Oct 4, 2023
Moving the gazelle logging util methods to a common util for all of the
cli (cli/core/pkg/logger/logger.go). For now this util is the basic go
`log` package, if we continue using this util for logging then we can
replace the underlying log library with anything in the future.

For now there are 2 undocumented env vars to configure logging level +
output file.
The log file + level is auto configured when used within `bazel test`
which has been used by the gazelle tests for quite some time.

Ref:
#428
#480
#290
#26
GitOrigin-RevId: dd5b7949117ec4298671945d7d0df1ba288b9d8b
jbedard added a commit that referenced this issue Oct 7, 2023
Moving the gazelle logging util methods to a common util for all of the
cli (cli/core/pkg/logger/logger.go). For now this util is the basic go
`log` package, if we continue using this util for logging then we can
replace the underlying log library with anything in the future.

For now there are 2 undocumented env vars to configure logging level +
output file.
The log file + level is auto configured when used within `bazel test`
which has been used by the gazelle tests for quite some time.

Ref:
#428
#480
#290
#26
GitOrigin-RevId: dd5b7949117ec4298671945d7d0df1ba288b9d8b
jbedard added a commit that referenced this issue Oct 7, 2023
Moving the gazelle logging util methods to a common util for all of the
cli (cli/core/pkg/logger/logger.go). For now this util is the basic go
`log` package, if we continue using this util for logging then we can
replace the underlying log library with anything in the future.

For now there are 2 undocumented env vars to configure logging level +
output file.
The log file + level is auto configured when used within `bazel test`
which has been used by the gazelle tests for quite some time.

Ref:
#428
#480
#290
#26
GitOrigin-RevId: dd5b7949117ec4298671945d7d0df1ba288b9d8b
jbedard added a commit that referenced this issue Oct 7, 2023
Moving the gazelle logging util methods to a common util for all of the
cli (cli/core/pkg/logger/logger.go). For now this util is the basic go
`log` package, if we continue using this util for logging then we can
replace the underlying log library with anything in the future.

For now there are 2 undocumented env vars to configure logging level +
output file.
The log file + level is auto configured when used within `bazel test`
which has been used by the gazelle tests for quite some time.

Ref:
#428
#480
#290
#26
GitOrigin-RevId: dd5b7949117ec4298671945d7d0df1ba288b9d8b
jbedard added a commit that referenced this issue Oct 9, 2023
Moving the gazelle logging util methods to a common util for all of the
cli (cli/core/pkg/logger/logger.go). For now this util is the basic go
`log` package, if we continue using this util for logging then we can
replace the underlying log library with anything in the future.

For now there are 2 undocumented env vars to configure logging level +
output file.
The log file + level is auto configured when used within `bazel test`
which has been used by the gazelle tests for quite some time.

Ref:
#428
#480
#290
#26
GitOrigin-RevId: dd5b7949117ec4298671945d7d0df1ba288b9d8b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: No status
Development

No branches or pull requests

1 participant