Skip to content

Conversation

@GitHK
Copy link
Contributor

@GitHK GitHK commented Sep 26, 2025

What do these changes do?

The resource/osparc/schedulerWorker.js is receiving events which is not able to process. I have disabled the logger put there to help with debugging (in case of issues) and replaced it with a comment.

Related issue/s

How to test

Dev-ops

@mergify
Copy link
Contributor

mergify bot commented Sep 26, 2025

🧪 CI Insights

Here's what we observed from your CI run for bc3f4b8.

🟢 All jobs passed!

But CI Insights is watching 👀

@GitHK GitHK changed the title 🎨 Added better log for error 🎨 Removed noisy logger Sep 29, 2025
@GitHK GitHK self-assigned this Sep 29, 2025
@GitHK GitHK added this to the Cheops milestone Sep 29, 2025
@GitHK GitHK requested review from alexpargon and Copilot September 29, 2025 10:43
@GitHK GitHK marked this pull request as ready for review September 29, 2025 10:43
@GitHK GitHK requested a review from odeimaiz as a code owner September 29, 2025 10:43
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes a noisy console log statement from the scheduler worker that was logging events it couldn't process. The change replaces the log statement with a comment suggesting how to re-enable logging for debugging purposes.

  • Removed console.log statement that was creating noise for unhandled worker events
  • Added explanatory comment for future debugging

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@GitHK GitHK enabled auto-merge (squash) October 1, 2025 07:29
@GitHK GitHK disabled auto-merge October 1, 2025 07:29
@GitHK GitHK enabled auto-merge (squash) October 6, 2025 13:21
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 6, 2025

@GitHK GitHK merged commit 6b176ba into ITISFoundation:master Oct 6, 2025
58 checks passed
@GitHK GitHK deleted the pr-osparc-debug-web-worker branch October 6, 2025 13:38
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.

6 participants