Skip to content

Add new queue message: JobStartedMessage#51

Merged
josecelano merged 0 commit into
mainfrom
issue-29-job-started-message
Feb 17, 2022
Merged

Add new queue message: JobStartedMessage#51
josecelano merged 0 commit into
mainfrom
issue-29-job-started-message

Conversation

@josecelano

@josecelano josecelano commented Feb 16, 2022

Copy link
Copy Markdown
Member

Issue: #29

@josecelano josecelano linked an issue Feb 16, 2022 that may be closed by this pull request
Add new queue message: JobStartedMessage #29
Closed
@github-actions

github-actions Bot commented Feb 16, 2022 •
edited
Loading

Copy link
Copy Markdown

MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 4 0 0.1s
✅ COPYPASTE jscpd yes no 1.29s
✅ CREDENTIALS secretlint yes no 0.81s
✅ GIT git_diff yes no 0.01s
✅ JAVASCRIPT standard 1 0 0 1.16s
✅ JSON eslint-plugin-jsonc 7 0 0 0.82s
✅ JSON jsonlint 7 0 1.59s
✅ JSON prettier 7 0 0 1.12s
✅ JSON v8r 7 0 6.85s
✅ MARKDOWN markdownlint 2 0 0 0.29s
✅ MARKDOWN markdown-link-check 2 0 1.34s
✅ MARKDOWN markdown-table-formatter 2 0 0 0.24s
✅ SPELL cspell 77 0 2.69s
✅ SPELL misspell 77 0 0 0.16s
✅ TYPESCRIPT eslint 51 0 0 11.49s
✅ YAML prettier 7 0 0 1.43s
✅ YAML v8r 7 0 6.94s
✅ YAML yamllint 7 0 0.23s

See errors details in artifact MegaLinter reports on CI Job page

@josecelano josecelano force-pushed the issue-29-job-started-message branch from 09445f8 to 3cabdc7 Compare February 16, 2022 16:56
@josecelano

josecelano commented Feb 16, 2022

Copy link
Copy Markdown
Member Author

WIP. I have to check that there is a StartedJobMessage before creating the FinishedJobMessage, instead of checking if there is a pending job.

@josecelano josecelano force-pushed the issue-29-job-started-message branch 3 times, most recently from e3d313e to 53e6093 Compare February 17, 2022 15:28
@josecelano josecelano merged commit b13e4c5 into main Feb 17, 2022
@josecelano josecelano deleted the issue-29-job-started-message branch February 17, 2022 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@da2ce7 da2ce7 Awaiting requested review from da2ce7

@yeraydavidrodriguez yeraydavidrodriguez Awaiting requested review from yeraydavidrodriguez

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add new queue message: JobStartedMessage

1 participant

@josecelano