Skip to content

feat: add max-length message display#1638

Open
dipinknair wants to merge 6 commits into
mainfrom
feat/message-clr
Open

feat: add max-length message display#1638
dipinknair wants to merge 6 commits into
mainfrom
feat/message-clr

Conversation

@dipinknair

Copy link
Copy Markdown
Contributor
  • upgrade clr loader
  • Add message max- length limit

@dipinknair dipinknair requested a review from a team as a code owner May 27, 2026 16:12
@github-actions github-actions Bot added dependencies Related with project dependencies enhancement New features or code improvements labels May 27, 2026
@codacy-production

codacy-production Bot commented May 27, 2026

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 2 high

Alerts:
⚠ 2 issues (≤ 0 issues of at least minor severity)

Results:
2 new issues

Category Results
ErrorProne 2 high

View in Codacy

🟢 Metrics 0 complexity

Metric Results
Complexity 0

View in Codacy

🟢 Coverage 78.57% diff coverage · -0.04% coverage variation

Metric Results
Coverage variation -0.04% coverage variation (-1.00%)
Diff coverage 78.57% diff coverage

View coverage diff in Codacy

Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (d764308) 3321 2680 80.70%
Head commit (40a1876) 3329 (+8) 2685 (+5) 80.65% (-0.04%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#1638) 14 11 78.57%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@dipinknair dipinknair marked this pull request as draft May 27, 2026 16:13
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label May 27, 2026
@github-actions

Copy link
Copy Markdown

The documentation for this pull request will be available at https://mechanical.docs.pyansys.com/pull/1638. Please allow some time for the documentation to be deployed.

@codecov

codecov Bot commented May 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 78.57143% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.65%. Comparing base (d764308) to head (40a1876).

❌ Your patch check has failed because the patch coverage (78.57%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1638      +/-   ##
==========================================
- Coverage   80.69%   80.65%   -0.05%     
==========================================
  Files          43       43              
  Lines        3321     3329       +8     
==========================================
+ Hits         2680     2685       +5     
- Misses        641      644       +3     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread src/ansys/mechanical/core/embedding/loader.py
@dipinknair dipinknair marked this pull request as ready for review June 11, 2026 16:50
@dipinknair dipinknair requested a review from koubaa June 11, 2026 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Related with project dependencies documentation Improvements or additions to documentation enhancement New features or code improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants