Conversation
|
The following people have not signed the Contributors License Agreement (CLA): Read the CLA in the link above and sign it by clicking below: You will receive a confirmation as soon as your signature is captured. Note Action Required: Repository maintainers with write access can trigger a recheck by commenting |
Not up to standards ⛔🔴 Issues
|
| Category | Results |
|---|---|
| CodeStyle | 50 minor |
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.
|
This probably should have been maint or chore or whatever we settled on, given that it isn't adding anything functionally. |
Right @seanpearsonuk. This should be chore. |
| transfer["connectivity: file/data transfer strategies"] -.-> sessions | ||
| transfer -.-> files | ||
| ``` | ||
|
|
There was a problem hiding this comment.
- Each bullet below mixes multiple, unrelated concerns, which we don't want.
- I can figure out the meanings of things only due to my PyFluent experience.
- Fields, a namespace, is used in the plural, so the context might easily be misinterpreted.
- Fields is defined by negation. A simpler definition is possible.
There was a problem hiding this comment.
I have refactored a bit here. The primary thing missing here was that it was not using code namespaces and thus would have wasted credits. That has been updated now.
Regarding the last point, I have updated it in this case, but in some instances a negation might be good idea to suggest agent what not to look for in a particular case.
Pleas have a look into the updated file.
Context
Add repository-specific agent guidance to improve accuracy and reduce unnecessary context loading, aligned with PyFluent’s restructured layout.
Change Summary
Rationale
Impact
Documentation only; no runtime behavior changes.