Skip to content

Merge PackageInstance and PackageTestConfig into one class #1920

@coderabbitai

Description

@coderabbitai

Context

During the review of PR #1843, the team discussed whether PackageInstance and PackageTestConfig should remain separate classes.

After discussion, the consensus is to merge them into a single class. The rationale:

  • The separation adds an additional layer of abstraction without clear concrete benefits at this stage
  • Even with more attributes in a merged class, we can use grouping and docstrings to maintain clarity
  • PackageInstance tracks intermediate states/artifacts while PackageTestConfig supplies input configs, but these purposes can coexist in one class

References

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions