Skip to content

Commit 9955045

Browse files
umbynosper1234
andauthored
Apply suggestions from code review
Co-authored-by: per1234 <[email protected]>
1 parent c265ae3 commit 9955045

5 files changed

+7
-11
lines changed

workflow-templates/publish-go-nightly-task.yml

+3-3
Original file line numberDiff line numberDiff line change
@@ -107,8 +107,8 @@ jobs:
107107
run: |
108108
cat > "${{ env.GON_CONFIG_PATH }}" <<EOF
109109
# See: https://github.com/mitchellh/gon#configuration-file
110-
source = [TODO_SOURCE_PATH]
111-
bundle_id = TODO_BUNDLE_ID
110+
source = ["${{ env.DIST_DIR }}/${{ env.PROJECT_NAME }}_osx_${{ matrix.artifact.name }}/${{ env.PROJECT_NAME }}"]
111+
bundle_id = "cc.arduino.${{ env.PROJECT_NAME }}"
112112
113113
sign {
114114
application_identity = "Developer ID Application: ARDUINO SA (7KT7ZWMCJT)"
@@ -139,7 +139,7 @@ jobs:
139139
run: |
140140
# GitHub's upload/download-artifact actions don't preserve file permissions,
141141
# so we need to add execution permission back until the action is made to do this.
142-
chmod +x "${{ env.PROJECT_NAME }}_${{ matrix.artifact.name }}/${{ env.PROJECT_NAME }}"
142+
chmod +x "${{ env.PROJECT_NAME }}_osx_${{ matrix.artifact.name }}/${{ env.PROJECT_NAME }}"
143143
# Use of an array here is required for globbing
144144
PACKAGE_FILENAME=(${{ env.PROJECT_NAME }}_nightly-*${{ matrix.artifact.path }})
145145
tar -czvf "$PACKAGE_FILENAME" \

workflow-templates/release-go-crosscompile-task.md

-2
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,6 @@ The following project-specific variables must be set/configured in `release-go-c
3636
- `PROJECT_NAME`
3737
- `AWS_PLUGIN_TARGET`
3838
- `GO_VERSION`: version of Go used for development of the project, use at least [GO 1.16 to be able to use 64-bit ARM architecture on macOS](https://tip.golang.org/doc/go1.16#ports)
39-
- `source`
40-
- `bundle_id`
4139

4240
#### Repository secrets
4341

workflow-templates/release-go-crosscompile-task.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -118,8 +118,8 @@ jobs:
118118
run: |
119119
cat > "${{ env.GON_CONFIG_PATH }}" <<EOF
120120
# See: https://github.com/mitchellh/gon#configuration-file
121-
source = [TODO_SOURCE_PATH]
122-
bundle_id = TODO_BUNDLE_ID
121+
source = ["${{ env.DIST_DIR }}/${{ env.PROJECT_NAME }}_osx_${{ matrix.artifact.name }}/${{ env.PROJECT_NAME }}"]
122+
bundle_id = "cc.arduino.${{ env.PROJECT_NAME }}"
123123
124124
sign {
125125
application_identity = "Developer ID Application: ARDUINO SA (7KT7ZWMCJT)"

workflow-templates/release-go-task.md

-2
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@ The following project-specific variables must be set in `release-go-task.yml`:
3535

3636
- `PROJECT_NAME`
3737
- `AWS_PLUGIN_TARGET`
38-
- `source`
39-
- `bundle_id`
4038

4139
#### Repository secrets
4240

workflow-templates/release-go-task.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -111,8 +111,8 @@ jobs:
111111
run: |
112112
cat > "${{ env.GON_CONFIG_PATH }}" <<EOF
113113
# See: https://github.com/mitchellh/gon#configuration-file
114-
source = [TODO_SOURCE_PATH]
115-
bundle_id = TODO_BUNDLE_ID
114+
source = ["${{ env.DIST_DIR }}/${{ env.PROJECT_NAME }}_osx_${{ matrix.artifact.name }}/${{ env.PROJECT_NAME }}"]
115+
bundle_id = "cc.arduino.${{ env.PROJECT_NAME }}"
116116
117117
sign {
118118
application_identity = "Developer ID Application: ARDUINO SA (7KT7ZWMCJT)"

0 commit comments

Comments
 (0)