-
-
Notifications
You must be signed in to change notification settings - Fork 712
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Automate HomeAssisstant releases #4369
Conversation
Add hassio config
Alternativ in der release |
Kannst Du aus den beiden PRs bitte einen machen? |
Würde ich weiß aber nicht wie. Geht das überhaupt mit der Browserversion? |
Ich habe jetzt alle commits in einen PR gepackt: packaging/hassio/config.json:
update workflows/release.json:
|
sonst funktioniert das nicht!
Ups, mir war nicht klar, dass Du das mit dem Browser machst. Jetzt auf jeden Fall perfekt, danke! |
@@ -108,8 +108,7 @@ jobs: | |||
# branch-name: master | |||
# author-name: evcc-io | |||
run: | | |||
export RELEASE_VERSION=$(echo ${{ env.GITHUB_REF }} | sed -e s#refs/tags/##) | |||
sed -e s#0.0.0#$RELEASE_VERSION# packaging/hassio/config.json | |||
sed -i -e s#0.0.0#$(echo ${{ env.GITHUB_REF }} | sed -e s#refs/tags/##)# packaging/hassio/config.json |
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.
Das geht nicht. config.json wird im nächsten step mit "version": "0.0.0"
kopiert. (getested)
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.
Yep, hab ich auch gesehen- das dürfte aber daran liegen, dass Du den branch angegeben hast. Geht es wenn Du den raus nimmst (so wie ich das hier nachgereicht habe)? Dann sollte die lokale Datei kopiert werden?
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.
bei mir wird der file nur richtig ausgeben bei:
run: |
export RELEASE_VERSION=$(echo ${{ github.ref }} | sed -e s#refs/tags/v##)
echo "$RELEASE_VERSION"
sed -e s#0.0.0#$RELEASE_VERSION# packaging/hassio/config.json
so geht es nicht:
run: |
sed -i -e s#0.0.0#$(echo ${{ env.GITHUB_REF }} | sed -e s#refs/tags/##)# packaging/hassio/config.json
CopyCat: Das geht nur mit branch bei mir, da meins "main" ist. Hier sollte es auch ohne branch gehen, da default "master" ist. Oder wo meinst du den "branch" ?
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.
wenn Du es mit sed machen willst wisrst du nicht 0.0.0 ersetzen können, sondern die zeile mit "version": "x.x.x" und dann mit "version": "release" ersetzen müssen. Aber den file musst du committen, oder aber direkt in das hassio-addon commiten. Am einfachsten ist es wenn hassio-addon im evcc branche wäre.
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.
wenn Du es mit sed machen willst wisrst du nicht 0.0.0 ersetzen können, sondern die zeile mit "version": "x.x.x" und dann mit "version": "release" ersetzen müssen.
Verstehe ich (leider) nicht. Ich hätte gerne Folgendes:
- Datei hier aktualisieren
- aktualisierte Datei ins hassio Repo committen
- kein Commit hier (wenn wir hier committen triggert das den nächsten Build der unnötig ist)
Tut nicht?
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.
Wenn wir die Logik erstmal fürs Nightly bauen könnten liesse sich das auch gefahrlos testen. hier müssen wir erst aufs ncähste Release warten...
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.
Ich teste das im hintergrund, mit meine repos. Mit sed geht es leider nicht (noch nicht).
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.
so der nightly vorschlag ist da: #4394
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.
Info:
The simplest option is to clone the target repo, copy the files into the target repo, use the git commandline to stage the files and then commit them. Add the code below in a script step
run: |
git clone https://.:${{ secrets.GITHUB_TOKEN }}@github.com/project target
rm -rf everything but the .git directory
copy source\files target
cd target
git add . #(add -A)
git diff-index --quiet HEAD || git commit -m "Automatic publish from github.com/project"
git push target master
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.
Add hassio config