Skip to content

Conversation

@haitaohuang
Copy link
Contributor

@haitaohuang haitaohuang commented Nov 27, 2025

No description provided.

@jyao1
Copy link
Contributor

jyao1 commented Nov 27, 2025

Comment: Please split this PR to two separated PRs.

  1. Add macro.
  2. fix the log.
    They are independent tasks.

Question: What is expectation for "log::error!" now?
I notice some of them having the "\n", but some not.

@haitaohuang haitaohuang force-pushed the intel_tracking branch 2 times, most recently from 0ed3a86 to a07f6f6 Compare November 27, 2025 02:21
@haitaohuang haitaohuang changed the title remove extra error log and add "\n" remove extra error log Nov 27, 2025
@haitaohuang
Copy link
Contributor Author

Comment: Please split this PR to two separated PRs.

  1. Add macro.
  2. fix the log.
    They are independent tasks.

Question: What is expectation for "log::error!" now? I notice some of them having the "\n", but some not.

Will only fix log for this PR.

@haitaohuang haitaohuang force-pushed the intel_tracking branch 2 times, most recently from d5b47af to c17088c Compare November 27, 2025 02:48
@haitaohuang haitaohuang marked this pull request as ready for review November 27, 2025 02:49
@haitaohuang haitaohuang requested a review from jyao1 as a code owner November 27, 2025 02:49
@jyao1 jyao1 linked an issue Nov 27, 2025 that may be closed by this pull request
@jyao1
Copy link
Contributor

jyao1 commented Nov 27, 2025

@haitaohuang please resolve conflict.

@haitaohuang
Copy link
Contributor Author

I'll rebase

@jyao1
Copy link
Contributor

jyao1 commented Nov 27, 2025

@haitaohuang , please do not use "merge direct" (Merge branch 'main' into intel_tracking) is not acceptable.

Please always use "rebase and merge".

@haitaohuang
Copy link
Contributor Author

done rebasing.

@jyao1 jyao1 merged commit 529c032 into intel:main Nov 27, 2025
51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unexpected error log appears with debug image

2 participants