Skip to content

Conversation

da2ce7
Copy link
Contributor

@da2ce7 da2ce7 commented Apr 5, 2022

Feature: Json Metadata as Commit Body

Create Schema Enforced Json Message the Commit Body.
Includes:

  • Namespace.
  • Version.
  • Custom Payload.

May be safely expanded in future by expanding the schema and issuing a new version.

closes #24

@github-actions
Copy link

github-actions bot commented Apr 5, 2022

MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 5 0 0.2s
✅ COPYPASTE jscpd yes no 1.91s
✅ CREDENTIALS secretlint yes no 1.04s
✅ GIT git_diff yes no 0.02s
✅ JAVASCRIPT standard 1 0 0 1.5s
✅ JSON eslint-plugin-jsonc 7 0 0 1.01s
✅ JSON jsonlint 7 0 1.98s
✅ JSON prettier 7 0 0 0.72s
✅ JSON v8r 7 0 7.84s
⚠️ MARKDOWN markdownlint 2 0 1 0.4s
✅ MARKDOWN markdown-link-check 2 0 5.02s
✅ MARKDOWN markdown-table-formatter 2 0 0 0.34s
✅ SPELL cspell 81 0 4.36s
✅ SPELL misspell 81 0 0 0.19s
✅ TYPESCRIPT eslint 54 0 0 14.62s
✅ YAML prettier 8 0 0 1.09s
✅ YAML v8r 8 0 10.43s
✅ YAML yamllint 8 0 0.3s

See errors details in artifact MegaLinter reports on CI Job page

Copy link
Collaborator

@yeraydavidrodriguez yeraydavidrodriguez left a comment

Choose a reason for hiding this comment

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

Checked that the changes are the same as in the non-squashed branch.
Thank you, @da2ce7 . I did a squash of the last commits in the other branch, but I'm afraid I made some mistakes (I'm not extremely good at solving [code] conflicts), had to force-push a few times and the branch got slightly messy.
This new branch is very much cleaner.

@github-actions
Copy link

github-actions bot commented Apr 13, 2022

MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 5 0 0.19s
✅ COPYPASTE jscpd yes no 1.81s
✅ CREDENTIALS secretlint yes no 0.97s
✅ GIT git_diff yes no 0.02s
✅ JAVASCRIPT standard 1 0 0 1.45s
✅ JSON eslint-plugin-jsonc 7 0 0 1.01s
✅ JSON jsonlint 7 0 1.89s
✅ JSON prettier 7 0 0 0.7s
✅ JSON v8r 7 0 8.12s
✅ MARKDOWN markdownlint 2 0 0 0.39s
✅ MARKDOWN markdown-link-check 2 0 3.14s
✅ MARKDOWN markdown-table-formatter 2 0 0 0.29s
✅ SPELL cspell 81 0 3.61s
✅ SPELL misspell 81 0 0 0.2s
✅ TYPESCRIPT eslint 54 0 0 14.17s
✅ YAML prettier 8 0 0 1.03s
✅ YAML v8r 8 0 9.53s
✅ YAML yamllint 8 0 0.31s

See errors details in artifact MegaLinter reports on CI Job page

@da2ce7 da2ce7 force-pushed the issue-24-add-json-metadata-commit-body-rebase branch from 47a8399 to ff1e70c Compare April 13, 2022 08:49
@da2ce7
Copy link
Contributor Author

da2ce7 commented Apr 13, 2022

@yeraydavidrodriguez rebased 👍

@yeraydavidrodriguez
Copy link
Collaborator

@yeraydavidrodriguez rebased 👍

Thanks @da2ce7 !

@da2ce7 da2ce7 changed the title Issue 24 Add json Metadata Commit Body (Squashed) Issue 24: Json Metadata as Commit Body Apr 13, 2022
@da2ce7 da2ce7 linked an issue Apr 13, 2022 that may be closed by this pull request
@da2ce7 da2ce7 force-pushed the issue-24-add-json-metadata-commit-body-rebase branch from ff1e70c to 3da3fd4 Compare April 13, 2022 11:11
Create Schema Enforced Json Message the Commit Body.
Includes:
- Namespace.
- Version.
- Custom Payload.

May be safely expanded in future by expanding the schema and issuing a new version.

(Implements Issue: #24)
@da2ce7 da2ce7 force-pushed the issue-24-add-json-metadata-commit-body-rebase branch from 3da3fd4 to 5e497bc Compare April 13, 2022 18:11
@da2ce7
Copy link
Contributor Author

da2ce7 commented Apr 13, 2022

@yeraydavidrodriguez rebased and squashed.

@da2ce7 da2ce7 changed the title Issue 24: Json Metadata as Commit Body Issue #24: Json Metadata as Commit Body Apr 13, 2022
@yeraydavidrodriguez yeraydavidrodriguez merged commit 63b97c5 into main Apr 18, 2022
@yeraydavidrodriguez yeraydavidrodriguez deleted the issue-24-add-json-metadata-commit-body-rebase branch April 18, 2022 07:04
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.

Use a json commit body to include queue metadata
3 participants