check styling -do not merge

[pawelrutkaq]

No description provided.

[Copilot]

Pull request overview

This PR appears to be a styling-focused change in the COM proxy method implementation, adjusting a local variable name inside queue-slot selection logic.

Changes:

• Renamed a local boolean in DetermineNextAvailableQueueSlot from all_inactive to all_inactive_.
• Updated the corresponding conditional to use the renamed variable.

---

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

[pawelrutkaq]

Checks: >
  -*,
  readability-identifier-naming

CheckOptions:
  - key: readability-identifier-naming.LocalVariableCase
    value: lower_case

  - key: readability-identifier-naming.LocalVariablePrefix
    value: ""

  - key: readability-identifier-naming.LocalVariableSuffix
    value: ""

  - key: readability-identifier-naming.LocalVariableIgnoredRegexp
    value: "^_$"

@4og @LittleHuba @FScholPer ie above config should catch such a naming violation. There is plenty of readability classifiers to describe styling.

[limdor]

Checks: >
  -*,
  readability-identifier-naming

CheckOptions:
  - key: readability-identifier-naming.LocalVariableCase
    value: lower_case

  - key: readability-identifier-naming.LocalVariablePrefix
    value: ""

  - key: readability-identifier-naming.LocalVariableSuffix
    value: ""

  - key: readability-identifier-naming.LocalVariableIgnoredRegexp
    value: "^_$"

@4og @LittleHuba @FScholPer ie above config should catch such a naming violation. There is plenty of readability classifiers to describe styling.

@pawelrutkaq what are you trying to achieve here? We are aware that it can be checked like that but it would require a lot of changes including breaking API and it was postponed for the moment.
#392 (comment)

[pawelrutkaq]

This is just the thing that we discussed at FT sync meeting today (a need for a common code style in score) PR to be closed probably in max few day.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. If this PR is still relevant, please leave a comment or push new changes to keep it open.