Jlc/strain dispatcher action #92
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
johanseto
force-pushed
the
jlc/strain-dispatcher-action
branch
6 times, most recently
from
7eb4b7e
to
a45bc52
Compare
andrey-canon
reviewed
Sep 1, 2023
| token: ${{ secrets.NELP_PAT_TEST }} | ||
| repository: johanv26/the-strain-tester | ||
| event-type: strain-update-pr | ||
| client-payload: '{"repo": "${{ github.repository }}", "ref": "${{ github.ref }}", "sha": "${{ github.sha }}", "property_json_path": "DISTRO_EOX_NELP_DPKG.version", "branch": "${{ github.ref_name }}"}' |
There was a problem hiding this comment.
I prefer individual settings, this json could be messy
johanseto
force-pushed
the
jlc/strain-dispatcher-action
branch
15 times, most recently
from
b9c1ee6
to
e519b24
Compare
johanseto
force-pushed
the
jlc/strain-dispatcher-action
branch
6 times, most recently
from
9caa4f6
to
62b967e
Compare
johanseto
force-pushed
the
jlc/strain-dispatcher-action
branch
15 times, most recently
from
79638a3
to
cca7c16
Compare
|
@johanv26 is this ready for review ? |
johanseto
force-pushed
the
jlc/strain-dispatcher-action
branch
from
cca7c16
to
2074f36
Compare
Right now, yes |
andrey-canon
approved these changes
Sep 7, 2023
johanseto
force-pushed
the
jlc/strain-dispatcher-action
branch
from
2074f36
to
7b18de7
Compare
feat: test some references
chore: add repo to paylod
chore: test my local repo that has master
feat: set multiline operator
chore: try to sent de pr
chore: launch enviroment in payload
feat: set the build_env
chore: add more interesting ref data
fix: solve typo in json
feat: remove / from repo path
chore: set correct head-ref without slaches
chore: send strain path
chore: return to send data withou changes
chore: debug dispathcer
chore: set last log commit
chore: set no merges
chore: checkout repo only in the main sha
chore: set log commit differnet cmd
this is somethind,a,,
chore: set the correct of the head chad
use head sha instead of merge sha
chore: fix typo
chore: fix another typo
feat: add condition for built
chore: print all
chore: crazy conditional | for if
The if key doesnt need the `{{}}`.
https://github.com/orgs/community/discussions/25641
https://hungvu.tech/advanced-github-actions-conditional-workflow
chore: set full logger
feat: set better name of variables
chore: fix missing `''`
fix: typo of `:`
fix: remove unneccesary log
build: test push image
fix: add correct property value
build: 2test push image
build: 3test push image
refactor: remove strain_env in payload
chore: always built images in sync
chore: set property via repo vars
chore: refactor strain property
chore: remove commit analysis
chore: run in each pull_request
chore: change order of condition of the action
feat: use base PAT
fix: wrong syntaxis of the if
feat: configure correct repo of strain
chore: remove personal token
use token of the action
feat: use ROBONEXT_PAT
johanseto
force-pushed
the
jlc/strain-dispatcher-action
branch
from
7b18de7
to
df523dc
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Describe what this pull request changes, and why. Include implications for people using this change.
Useful information to include:
changes.
Testing instructions
Before
After
Additional information
Include anything else that will help reviewers and consumers understand the change.
Checklist for Merge