Skip to content

Remove leftover debug console.log from preview reducer#3824

Merged
raclim merged 2 commits intoprocessing:developfrom
aashu2006:remove-debug-console-log-preview
Feb 3, 2026
Merged

Remove leftover debug console.log from preview reducer#3824
raclim merged 2 commits intoprocessing:developfrom
aashu2006:remove-debug-console-log-preview

Conversation

@aashu2006
Copy link
Contributor

Changes:

  • Removes a leftover console.log used for debugging in the preview reducer.
  • No functional changes.

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • has no typecheck errors (npm run typecheck)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123
  • meets the standards outlined in the accessibility guidelines

Copy link
Contributor

@yugalkaushik yugalkaushik left a comment

Choose a reason for hiding this comment

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

Hi @aashu2006 thank you for the PR, just a small question What is the need for removing it?

@aashu2006
Copy link
Contributor Author

Hi @yugalkaushik, thanks for reviewing!
I noticed this console.log was likely left from debugging. It runs every time the preview updates and doesn’t affect any functionality, so it just adds extra noise in the console. I removed it as a small cleanup.

@aashu2006
Copy link
Contributor Author

Please let me know @yugalkaushik if anything is needed from my side 🙂
Happy to help!

Copy link
Collaborator

@raclim raclim left a comment

Choose a reason for hiding this comment

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

Thanks everyone!

@raclim raclim merged commit 347e9ff into processing:develop Feb 3, 2026
1 check passed
@aashu2006
Copy link
Contributor Author

Thanks! Appreciate the review and collaboration 🙂

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.

4 participants