Skip to content

Conversation

@mrmundt
Copy link
Contributor

@mrmundt mrmundt commented Oct 1, 2025

Fixes NA

Summary/Motivation:

There was a new typos version released, and it actually had quite a few fixes. There were a couple false positives that have been added to the toml file.

Changes proposed in this PR:

  • Fix typos
  • Mark false positives

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the license terms described in the LICENSE.txt file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

@codecov
Copy link

codecov bot commented Oct 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.06%. Comparing base (c42c0c3) to head (8faf51d).
⚠️ Report is 39 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1675      +/-   ##
==========================================
- Coverage   77.07%   77.06%   -0.01%     
==========================================
  Files         395      395              
  Lines       62759    62759              
  Branches    10234    10234              
==========================================
- Hits        48369    48366       -3     
- Misses      11978    11982       +4     
+ Partials     2412     2411       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ksbeattie ksbeattie added the Priority:High High Priority Issue or PR label Oct 2, 2025
Copy link
Member

@ksbeattie ksbeattie left a comment

Choose a reason for hiding this comment

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

LGTM

@sufikaur sufikaur merged commit 0ca2acb into IDAES:main Oct 2, 2025
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority:High High Priority Issue or PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants