Skip to content

Conversation

@moijes12
Copy link
Contributor

Changes Introduced

This change updates the job model to use a fixed set of TextChoices similar to an Enum. This ensures that the operations such as comparisons of Job states involve fixed string constants and are not susceptible to any typos.

Testing

  • No new tests added

This change updates the job model to use a fixed set of TextChoices
similar to an Enum. This ensures that the operations such as
comparisons of Job states involve fixed string constants and are
not susceptible to any typos.
@moijes12 moijes12 marked this pull request as draft December 18, 2025 12:17
@moijes12 moijes12 marked this pull request as ready for review December 18, 2025 12:43
Copy link
Collaborator

@camd camd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't try running with this, but it looks good to me! Great idea to do this.

@moijes12
Copy link
Contributor Author

@Archaeopteryx Please can you review this

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.

2 participants