Added seconds to WorkItemMigrationProcessor estimation

[sbouchexbellomie-Philips]

Issue:

Test Case][Complete: 2124/3118][sid:174211|Rev: 26][tid: null | Average time of 3.221 per work item and 0 hours 53 minutes 12 seconds estimated to completion

Unit is missing in the message

Added seconds

Summary by CodeRabbit

• Chores
  • Updated internal logging message formatting for migration processing.

[CLAassistant]

All committers have signed the CLA.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8936a733-426d-404f-8aba-7452bcaf54da

📥 Commits

Reviewing files that changed from the base of the PR and between b43c267 and 5af6b70.

📒 Files selected for processing (1)

• src/MigrationTools.Clients.TfsObjectModel/Processors/TfsWorkItemMigrationProcessor.cs

---

Walkthrough

A single-line informational log message in the work-item migration processor is updated to clarify how average processing time and estimated remaining time are reported to operators.

Changes

Logging Format Update

Layer / File(s)	Summary
Per-work-item timing log format src/MigrationTools.Clients.TfsObjectModel/Processors/TfsWorkItemMigrationProcessor.cs	The TraceWriteLine format string at line 698 adjusts phrasing to improve clarity of the average time and completion ETA reporting without changing the underlying data being logged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A message, once fuzzy, now shines crystal bright,
Operators glance and they know what's right. ✨
One line rewoven, the meaning takes flight,
Clarity triumphs—what a pragmatic delight. 📝

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly addresses the core change: adding seconds unit to the WorkItemMigrationProcessor's time estimation output, which matches the implementation in the summary.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[MrHinsh]

@sbouchexbellomie-Philips this looks good, can you resign the CLA?

[sbouchexbellomie-Philips]

@MrHinsh already signed