Skip to content

Commit fa4173d

Browse files
committed
Phase 06-01 ensure DEPS gets updated
1 parent 011d2c0 commit fa4173d

2 files changed

Lines changed: 6 additions & 4 deletions

File tree

.github/workflows/java-publish-maven.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -172,13 +172,16 @@ jobs:
172172
run: |
173173
VERSION="${{ steps.versions.outputs.release_version }}"
174174
175-
# Update version in README.md (supports any version qualifier like -java.N, -java-preview.N, -beta-java.N)
175+
# Update release version in README.md (supports any version qualifier like -java.N, -java-preview.N, -beta-java.N)
176176
sed -i "s|<version>[0-9][0-9]*\.[0-9][0-9]*\.[0-9][0-9]*\(-[a-z][a-z0-9-]*\.[0-9][0-9]*\)*</version>|<version>${VERSION}</version>|g" README.md
177177
sed -i "s|copilot-sdk-java:[0-9][0-9]*\.[0-9][0-9]*\.[0-9][0-9]*\(-[a-z][a-z0-9-]*\.[0-9][0-9]*\)*|copilot-sdk-java:${VERSION}|g" README.md
178178
179-
# Update snapshot version in README.md
179+
# Update snapshot versions in README.md (must run AFTER release version seds
180+
# because the release copilot-sdk-java: pattern partially matches inside snapshot
181+
# strings — the snapshot-specific seds override with the correct DEV_VERSION)
180182
DEV_VERSION="${{ steps.versions.outputs.dev_version }}"
181183
sed -i "s|<version>[0-9][0-9]*\.[0-9][0-9]*\.[0-9][0-9]*\(-[a-z][a-z0-9-]*\.[0-9][0-9]*\)*-SNAPSHOT</version>|<version>${DEV_VERSION}</version>|g" README.md
184+
sed -i "s|copilot-sdk-java:[0-9][0-9]*\.[0-9][0-9]*\.[0-9][0-9]*\(-[a-z][a-z0-9-]*\.[0-9][0-9]*\)*-SNAPSHOT|copilot-sdk-java:${DEV_VERSION}|g" README.md
182185
183186
# Update version in jbang-example.java
184187
sed -i "s|copilot-sdk-java:[0-9][0-9]*\.[0-9][0-9]*\.[0-9][0-9]*\(-[a-z][a-z0-9-]*\.[0-9][0-9]*\)*|copilot-sdk-java:${VERSION}|g" jbang-example.java
@@ -312,4 +315,3 @@ jobs:
312315
echo "Triggered deploy-site.yml on github/copilot-sdk-java for version ${VERSION}" >> $GITHUB_STEP_SUMMARY
313316
env:
314317
GITHUB_TOKEN: ${{ secrets.JAVA_RELEASE_GITHUB_TOKEN }}
315-

80-java-monorepo-add-01-remove-before-merge/dd-2989727-move-java-to-monorepo-plan.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ The software that currently resides in https://github.com/github/copilot-sdk-jav
179179

180180
1. ✅ Re-activate JaCoCo after Steve's https://github.com/github/copilot-sdk/pull/1492
181181

182-
2. Bring changelog concept to parity compared between standalone and monorepo See https://github.com/github/copilot-sdk-partners/issues/122
182+
2. Bring changelog concept to parity compared between standalone and monorepo See https://github.com/github/copilot-sdk-partners/issues/122
183183

184184
3. Deep review of release process
185185

0 commit comments

Comments
 (0)