remote_door: Ignore new pylint warning

[ldoktor]

for some reason updated pylint doesn't like NS_MODULE name even though it's top-level. Let's ignore it to fix CI.

Summary by CodeRabbit

• Chores
  • Updated linter configuration to acknowledge coding style guidelines with no impact on functionality.

[coderabbitai]

Walkthrough

The change adds pylint: disable=C0103 comments to NS_MODULE assignments in aexpect/remote_door.py. These annotations acknowledge pylint's naming convention warnings (C0103 represents invalid-name violations) without affecting any functional behavior or runtime operation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

• Single file with cosmetic linting comment additions
• No logic, functional, or behavioral modifications
• Homogeneous pattern of identical comment insertions

Pre-merge checks

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The pull request title "remote_door: Ignore new pylint warning" is directly aligned with the main change described in the raw summary. The changeset involves adding pylint: disable=C0103 comments to NS_MODULE assignments to suppress a new pylint warning, and the title accurately and concisely captures this primary objective. The title is specific enough to convey the file affected (remote_door) and the nature of the change (ignoring a pylint warning), without being vague or overly generic.

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0b40a4e and 42002e8.

📒 Files selected for processing (1)

• aexpect/remote_door.py (2 hunks)

🔇 Additional comments (3)

aexpect/remote_door.py (3)

80-80: Clean fix for new pylint warning.

Adding the pylint: disable=C0103 comment is the right approach here. The variable name NS_MODULE legitimately needs to be in SCREAMING_SNAKE_CASE to reflect its role as a module-path constant, despite the newer pylint version flagging it as invalid-name.

---

102-102: Consistent linting directive across exception handlers.

The disable comment is consistently applied to all three conditional assignments of NS_MODULE (Pyro5 case, Pyro4 fallback, and import failure case), which maintains code clarity and uniformity.

---

109-109: Complete coverage of pylint warnings.

All three assignments of NS_MODULE now have the appropriate pylint disable comment, ensuring CI will pass without requiring code structure changes.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[ldoktor]

CI passed, let's merge it to avoid CI failures.