Skip to content
This repository was archived by the owner on Sep 24, 2025. It is now read-only.
This repository was archived by the owner on Sep 24, 2025. It is now read-only.

[non-blocking] as a default we should avoid string manipulations in higher level packages. This should be a Name() method on the Job model. If we don't have the job model when calling functions like this, then we could also have a JobID type which has a Name method that does the string manipulation. #191

@tty47

Description

@tty47
          [non-blocking] as a default we should avoid string manipulations in higher level packages. This should be a `Name()` method on the `Job` model.  If we don't have the job model when calling functions like this, then we could also have a `JobID` type which has a `Name` method that does the string manipulation. 

this helps reduce the surface error for bugs, so we aren't doing this manipulations in multiple places.

Let's handle this in a follow up issue so that we can properly evaluate the best option.

Originally posted by @MSevey in #188 (comment)

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions