Skip to content

Commit 31af8da

Browse files
committed
Fix some issues with the GH build.
- Ensure duplicate LemMinX uber jar at root does not get packaged. - Include the universal vsix in the staged artifacts - Ensure VSIX artifact are named after the extension version - Publish the existing universal vsix, rather than from scratch Signed-off-by: Roland Grunberg <rgrunber@redhat.com>
1 parent a4654cc commit 31af8da

File tree

1 file changed

+12
-11
lines changed

1 file changed

+12
-11
lines changed

Diff for: .github/workflows/release.yml

+12-11
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ jobs:
5656
- name: Install dependencies
5757
run: npm install -g typescript "vsce" "ovsx"
5858
- run: echo "XML_SERVER_VERSION=$(cat package.json | jq -r .xmlServer.version)" >> $GITHUB_ENV
59+
- run: echo "EXT_VERSION=$(cat package.json | jq -r .version)" >> $GITHUB_ENV
5960
- name: Download LemMinX Server Uber Jar
6061
env:
6162
downloadLocation: https://github.com/redhat-developer/vscode-xml
@@ -70,15 +71,14 @@ jobs:
7071
- run: |
7172
mkdir server/
7273
if [ -e org.eclipse.lemminx*-uber.jar ]; then
73-
cp org.eclipse.lemminx*-uber.jar server/
74+
mv org.eclipse.lemminx*-uber.jar server/
7475
else
7576
cp ../staging/org.eclipse.lemminx*-uber.jar server/
7677
fi
7778
- name: Prepare Pre-Release
7879
if: ${{ github.event_name == 'schedule' || inputs.publishPreRelease == 'true' }}
7980
run: |
8081
npx gulp prepare_pre_release
81-
echo "EXT_VERSION=$(cat package.json | jq -r .version)" >> $GITHUB_ENV
8282
echo "publishPreReleaseFlag=--pre-release" >> $GITHUB_ENV
8383
- name: Download LemMinX Binary Artifacts
8484
uses: actions/download-artifact@v3
@@ -104,17 +104,17 @@ jobs:
104104
for tg in ${!targets[@]}; do
105105
cp ../staging/lemminx-${targets[${tg}]}* ./server
106106
cp ../staging/checksums/lemminx-${targets[${tg}]}.sha256 ./server
107-
vsce package ${{ env.publishPreReleaseFlag }} --target ${tg} -o vscode-xml-${tg}-${{ env.XML_SERVER_VERSION }}-${GITHUB_RUN_NUMBER}.vsix
107+
vsce package ${{ env.publishPreReleaseFlag }} --target ${tg} -o vscode-xml-${tg}-${{ env.EXT_VERSION }}-${GITHUB_RUN_NUMBER}.vsix
108108
rm ./server/lemminx-*
109109
done
110110
cp ../staging/checksums/lemminx-*.sha256 ./server
111-
vsce package -o vscode-xml-${{ env.XML_SERVER_VERSION }}-${GITHUB_RUN_NUMBER}.vsix
111+
vsce package -o vscode-xml-${{ env.EXT_VERSION }}-${GITHUB_RUN_NUMBER}.vsix
112112
ls -lash *.vsix
113113
- name: Upload VSIX Artifacts
114114
uses: actions/upload-artifact@v2
115115
with:
116116
name: vscode-xml
117-
path: vscode-xml-*-${{ env.XML_SERVER_VERSION }}-${{github.run_number}}.vsix
117+
path: vscode-xml*-${{ env.EXT_VERSION }}-${{github.run_number}}.vsix
118118
if-no-files-found: error
119119
- name: Upload LemMinX Uber Jar
120120
uses: actions/upload-artifact@v2
@@ -127,10 +127,10 @@ jobs:
127127
uses: "marvinpinto/action-automatic-releases@919008cf3f741b179569b7a6fb4d8860689ab7f0"
128128
with:
129129
repo_token: "${{ secrets.GITHUB_TOKEN }}"
130-
automatic_release_tag: "${{ env.XML_SERVER_VERSION }}"
130+
automatic_release_tag: "${{ env.EXT_VERSION }}"
131131
draft: true
132132
files: |
133-
vscode-xml-*-${{ env.XML_SERVER_VERSION }}-${{ github.run_number }}.vsix
133+
vscode-xml*-${{ env.EXT_VERSION }}-${{ github.run_number }}.vsix
134134
../staging/zips/lemminx-*
135135
../staging/checksums/lemminx-*
136136
release-job:
@@ -147,6 +147,7 @@ jobs:
147147
- name: Install dependencies
148148
run: npm install -g typescript "vsce" "ovsx"
149149
- run: echo "XML_SERVER_VERSION=$(cat package.json | jq -r .xmlServer.version)" >> $GITHUB_ENV
150+
- run: echo "EXT_VERSION=$(cat package.json | jq -r .version)" >> $GITHUB_ENV
150151
- name: Set the link to download the binary server
151152
env:
152153
downloadLocation: https://github.com/redhat-developer/vscode-xml
@@ -170,14 +171,14 @@ jobs:
170171
- name: Publish to VS Code Marketplace
171172
if: ${{ github.event_name == 'schedule' || inputs.publishToMarketPlace == 'true' || inputs.publishPreRelease == 'true' }}
172173
run: |
173-
for platformVsix in vscode-xml/vscode-xml-*-${{ env.XML_SERVER_VERSION }}-${GITHUB_RUN_NUMBER}.vsix; do
174+
for platformVsix in vscode-xml/vscode-xml-*-${{ env.EXT_VERSION }}-${GITHUB_RUN_NUMBER}.vsix; do
174175
vsce publish -p ${{ secrets.VSCODE_MARKETPLACE_TOKEN }} --packagePath ${platformVsix}
175176
done
176-
vsce publish -p ${{ secrets.VSCODE_MARKETPLACE_TOKEN }} ${{ env.publishPreReleaseFlag }}
177+
vsce publish -p ${{ secrets.VSCODE_MARKETPLACE_TOKEN }} --packagePath vscode-xml/vscode-xml-${{ env.EXT_VERSION }}-${GITHUB_RUN_NUMBER}.vsix ${{ env.publishPreReleaseFlag }}
177178
- name: Publish to OpenVSX Registry
178179
if: ${{ github.event_name == 'schedule' || inputs.publishToOVSX == 'true' || inputs.publishPreRelease == 'true' }}
179180
run: |
180-
for platformVsix in vscode-xml/vscode-xml-*-${{ env.XML_SERVER_VERSION }}-${GITHUB_RUN_NUMBER}.vsix; do
181+
for platformVsix in vscode-xml/vscode-xml-*-${{ env.EXT_VERSION }}-${GITHUB_RUN_NUMBER}.vsix; do
181182
ovsx publish -p ${{ secrets.OVSX_MARKETPLACE_TOKEN }} --packagePath ${platformVsix}
182183
done
183-
ovsx publish -p ${{ secrets.OVSX_MARKETPLACE_TOKEN }} ${{ env.publishPreReleaseFlag }}
184+
ovsx publish -p ${{ secrets.OVSX_MARKETPLACE_TOKEN }} --packagePath vscode-xml/vscode-xml-${{ env.EXT_VERSION }}-${GITHUB_RUN_NUMBER}.vsix ${{ env.publishPreReleaseFlag }}

0 commit comments

Comments
 (0)