Skip to content

Commit 8df699e

Browse files
author
j-zhangyiyuan
committed
fix: Java README version stuck at 1.0.5-01; release sed regex can't match numeric qualifiers
The qualifier group \(-[a-z]...)\ requires qualifiers to start with a letter. Numeric-only qualifiers like \-01\ don't match, so the Maven snippet was never updated past 1.0.5-01. Fix the regex to accept any alphanumeric qualifier start character: \(-[a-zA-Z0-9][a-zA-Z0-9.-]*)\ Also update the stale README versions to 1.0.6 (latest release). Closes #2209
1 parent 72de60f commit 8df699e

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -200,9 +200,9 @@ jobs:
200200
run: |
201201
VERSION="${{ steps.versions.outputs.release_version }}"
202202
203-
# Update release version in README.md (supports any version qualifier like -java.N, -java-preview.N, -beta-java.N)
204-
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
205-
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
203+
# Update release version in README.md (supports any version qualifier like -01, -java.N, -preview.N)
204+
sed -i "s|<version>[0-9][0-9]*\.[0-9][0-9]*\.[0-9][0-9]*\(-[a-zA-Z0-9][a-zA-Z0-9.-]*\)*</version>|<version>${VERSION}</version>|g" README.md
205+
sed -i "s|copilot-sdk-java:[0-9][0-9]*\.[0-9][0-9]*\.[0-9][0-9]*\(-[a-zA-Z0-9][a-zA-Z0-9.-]*\)*|copilot-sdk-java:${VERSION}|g" README.md
206206
207207
# Update snapshot versions in README.md (must run AFTER release version seds
208208
# because the release copilot-sdk-java: pattern partially matches inside snapshot

java/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,14 +32,14 @@ Replace `${copilot.sdk.version}` with the latest release from Maven Central.
3232
<dependency>
3333
<groupId>com.github</groupId>
3434
<artifactId>copilot-sdk-java</artifactId>
35-
<version>1.0.5-01</version>
35+
<version>1.0.6</version>
3636
</dependency>
3737
```
3838

3939
### Gradle
4040

4141
```groovy
42-
implementation 'com.github:copilot-sdk-java:1.0.9-preview.2-01'
42+
implementation 'com.github:copilot-sdk-java:1.0.6'
4343
```
4444

4545
#### Snapshot Builds

0 commit comments

Comments
 (0)