Create logging util system in ng-dev by josephperrott · Pull Request #37192 · angular/angular · GitHub
Skip to content

Create logging util system in ng-dev - #37192

Closed
josephperrott wants to merge 2 commits into
angular:masterfrom
josephperrott:ng-dev-logging-common
Closed

josephperrott wants to merge 2 commits into
angular:masterfrom
josephperrott:ng-dev-logging-common

Conversation

@josephperrott

Copy link
Copy Markdown
Member

See individual commits.

@josephperrott josephperrott added state: WIP action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release area: dev-infra Issues related to Angular's own dev infra (build, test, CI, releasing) labels May 18, 2020
@josephperrott
josephperrott requested a review from devversion May 18, 2020 23:02
@ngbot ngbot Bot added this to the needsTriage milestone May 18, 2020
@josephperrott
josephperrott force-pushed the ng-dev-logging-common branch from 60f0fa0 to 6a6c71c Compare May 18, 2020 23:07
Comment thread dev-infra/cli.ts Outdated
Comment thread dev-infra/cli.ts Outdated
@josephperrott josephperrott changed the title Creat logging util system in ng-dev Create logging util system in ng-dev May 19, 2020
@josephperrott
josephperrott force-pushed the ng-dev-logging-common branch 2 times, most recently from f45c471 to 6560fd3 Compare May 19, 2020 18:34
@josephperrott josephperrott added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer state: WIP labels May 19, 2020
@josephperrott
josephperrott marked this pull request as ready for review May 19, 2020 22:41

@devversion devversion left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When you get a chance, could you update the commit message? It seems to still mention the initial design where the --log-level flag was created.

Comment thread dev-infra/utils/console.ts Outdated
Comment thread dev-infra/utils/console.ts Outdated
Comment thread dev-infra/utils/console.ts Outdated
Comment thread dev-infra/utils/console.ts Outdated
@devversion devversion added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label May 20, 2020
…g levels

Creates common logging functions at different levels. Allows for providing
logging statements which are actually printed to the console based on the
LOG_LEVEL environment variable.
@josephperrott
josephperrott force-pushed the ng-dev-logging-common branch from 6560fd3 to ae646cb Compare May 20, 2020 15:08
@josephperrott josephperrott removed the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label May 20, 2020
@josephperrott
josephperrott requested a review from devversion May 20, 2020 15:08
Comment thread dev-infra/utils/console.ts
@kara kara closed this in 7db177d May 20, 2020
kara pushed a commit that referenced this pull request May 20, 2020
…g levels (#37192)

Creates common logging functions at different levels. Allows for providing
logging statements which are actually printed to the console based on the
LOG_LEVEL environment variable.

PR Close #37192
@angular-automatic-lock-bot

Copy link
Copy Markdown

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Jun 20, 2020
profanis pushed a commit to profanis/angular that referenced this pull request Sep 5, 2020
…g levels (angular#37192)

Creates common logging functions at different levels. Allows for providing
logging statements which are actually printed to the console based on the
LOG_LEVEL environment variable.

PR Close angular#37192
@josephperrott
josephperrott deleted the ng-dev-logging-common branch August 31, 2023 17:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: dev-infra Issues related to Angular's own dev infra (build, test, CI, releasing) cla: yes target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants