Skip to content

Standardize MissingAttributeError in top-level namespace#1747

Closed
Dxboy266 wants to merge 1 commit intocclib:masterfrom
Dxboy266:standardize-missing-attribute-error
Closed

Standardize MissingAttributeError in top-level namespace#1747
Dxboy266 wants to merge 1 commit intocclib:masterfrom
Dxboy266:standardize-missing-attribute-error

Conversation

@Dxboy266
Copy link

@Dxboy266 Dxboy266 commented Mar 7, 2026

Closes #1746

Summary

  • Add MissingAttributeError to cclib/__init__.py as the single canonical definition
  • Remove duplicate definitions from 4 files:
    • cclib/io/filewriter.py
    • cclib/bridge/cclib2pyquante.py
    • cclib/bridge/cclib2pyscf.py
    • cclib/method/calculationmethod.py
  • Update test imports to use top-level import

Changes

10 files changed, 14 insertions(+), 13 deletions(-)

This is a simple refactoring that consolidates the MissingAttributeError exception class into a single location for consistency and easier maintenance.

- Add MissingAttributeError to cclib/__init__.py
- Remove duplicate definitions from:
  - cclib/io/filewriter.py
  - cclib/bridge/cclib2pyquante.py
  - cclib/bridge/cclib2pyscf.py
  - cclib/method/calculationmethod.py
- Update test imports to use top-level import

Closes cclib#1746
@berquist
Copy link
Member

berquist commented Mar 7, 2026

While we develop a policy on AI-based contributions (#1749), I'm closing this as it is not in the intent of "good first issues" for them to be solved entirely by AI.

@berquist berquist closed this Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Standardize on MissingAttributeError

2 participants