-
Notifications
You must be signed in to change notification settings - Fork 1
WIP: Init project #2
base: master
Are you sure you want to change the base?
Conversation
this branch is supposed to contain the initialisation of the project
type JobStatus int | ||
|
||
// JobStatus implementation as enum | ||
const ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is aligned to the yabs/api, looks good
type RunnerStatus int | ||
|
||
// RunnerStatus implementation as enum | ||
const ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fforootd
and here the runner status
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i did reduce the api side to the following
enum Status {
UNKNOWN = 0;
ACTIVE = 1;
PAUSED = 2;
ONLINE = 3;
OFFLINE = 4;
}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this alignment with the gitlab-runner status conditions works for me so far 👍
No description provided.