Skip to content

Conversation

@agarny
Copy link
Contributor

@agarny agarny commented Dec 3, 2025

No description provided.

Copilot AI review requested due to automatic review settings December 3, 2025 21:58
Copy link

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 performs cleanup by fixing formatting issues in error messages. The changes correct two error messages by adding missing spaces and fixing the order of variable/component names in the messages.

Key changes:

  • Fixed missing spaces in error messages (e.g., 'environment'cannot'environment' cannot)
  • Corrected the order of variable and component names in one error message to properly show "variable 'X' in component 'Y'" instead of "variable 'Y' in component 'X'"
  • Updated version to 0.20251204.0

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/sed/sedchangeattribute.cpp Fixed spacing and corrected variable/component name order in two error message strings
tests/bindings/python/test_solver_coverage.py Updated expected error messages to match the corrected source code messages
tests/bindings/javascript/solver.coverage.test.js Updated expected error messages to match the corrected source code messages
tests/api/solver/coveragetests.cpp Updated expected error messages to match the corrected source code messages
VERSION.txt Bumped version number to 0.20251204.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Indeed, that version is not compatible with `sphinx-copybutton` and `sphinx-inline-tabs`.
@agarny agarny force-pushed the cleaning-up branch 4 times, most recently from 842d15e to 52a8aee Compare December 4, 2025 00:34
@agarny agarny merged commit a7b4459 into opencor:main Dec 4, 2025
24 checks passed
@agarny agarny deleted the cleaning-up branch December 4, 2025 01:58
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.

1 participant