Skip to content

fix(core): show all native logs for debug mode#31154

Merged
FrozenPandaz merged 1 commit intomasterfrom
improve-debug
May 9, 2025
Merged

fix(core): show all native logs for debug mode#31154
FrozenPandaz merged 1 commit intomasterfrom
improve-debug

Conversation

@FrozenPandaz
Copy link
Copy Markdown
Contributor

Current Behavior

TUI debug mode only shows native logs since the TUI launched.

Expected Behavior

TUI debug mode shows logs even before the TUI launched such as task hashing logs and more.

Related Issue(s)

Fixes #

@FrozenPandaz FrozenPandaz requested review from a team as code owners May 9, 2025 16:32
@vercel
Copy link
Copy Markdown

vercel Bot commented May 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview May 9, 2025 7:43pm

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented May 9, 2025

View your CI Pipeline Execution ↗ for commit 67c3625.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 33m 54s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 14s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 1s View ↗
nx documentation ✅ Succeeded 5m 37s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-09 20:25:58 UTC

Comment thread packages/nx/src/native/logger/mod.rs Outdated
let registry = tracing_subscriber::registry()
.with(stdout_layer)
.with(TuiTracingSubscriberLayer);
tui_logger::init_logger(tui_logger::LevelFilter::Trace).unwrap();
Copy link
Copy Markdown
Member

@Cammisuli Cammisuli May 9, 2025

Choose a reason for hiding this comment

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

expect("") or .ok() please. Probably .ok() since we don't want this to just break the whole run, and just ignore any Err(_)

@FrozenPandaz FrozenPandaz force-pushed the improve-debug branch 2 times, most recently from 743212b to dd39f06 Compare May 9, 2025 18:52
@FrozenPandaz FrozenPandaz enabled auto-merge (squash) May 9, 2025 19:03
@FrozenPandaz FrozenPandaz merged commit b620ea7 into master May 9, 2025
6 checks passed
@FrozenPandaz FrozenPandaz deleted the improve-debug branch May 9, 2025 20:26
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators May 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants