Skip to content

Conversation

@WilliamBergamin
Copy link
Contributor

These changes update the release process to support snapshot releases. It documents how to release a snapshot release and updates the release script to be less error prone.

Category

  • bolt (Bolt for Java)
  • bolt-{sub modules} (Bolt for Java - optional modules)
  • slack-api-client (Slack API Clients)
  • slack-api-model (Slack API Data Models)
  • slack-api-*-kotlin-extension (Kotlin Extensions for Slack API Clients)
  • slack-app-backend (The primitive layer of Bolt for Java)
  • Other

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.

@WilliamBergamin WilliamBergamin added this to the 1.45.5 milestone Oct 8, 2025
@WilliamBergamin WilliamBergamin self-assigned this Oct 8, 2025
@WilliamBergamin WilliamBergamin added enhancement M-T: A feature request for new functionality dependencies Pull requests that update a dependency file labels Oct 8, 2025
dir=`dirname $0`/..
release_version=`sed -n 's/<version>\([^\$]\..*\)<\/version>$/\1/p' < ${dir}/pom.xml`
dir=$(dirname "$0")/..
release_version=$(sed -n 's/^[[:space:]]*<version>\([^\$]\..*\)<\/version>[[:space:]]*$/\1/p' < "${dir}"/pom.xml)
Copy link
Contributor

@vegeris vegeris Oct 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIUC, the update to this command* removes spaces that were being added to the release_version variable?

Based on local testing I got

% release_version=`sed -n 's/<version>\([^\$]\..*\)<\/version>$/\1/p' < pom.xml`          
% echo $release_version                                                                
    1.45.5-SNAPSHOT

# vs.
% release_version=$(sed -n 's/^[[:space:]]*<version>\([^\$]\..*\)<\/version>[[:space:]]*$/\1/p' < pom.xml) 
% echo $release_version                                                                                    
1.45.5-SNAPSHOT

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes exactly 🚀

Copy link
Contributor

@vegeris vegeris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :) (Based on these docs)

@vegeris
Copy link
Contributor

vegeris commented Oct 8, 2025

The following artifacts could not be resolved: software.amazon.awssdk:bom-internal:pom:2.31.12

(From the test failure on the bolt project) Do we need to update the version of this dependency?

Actually, I thought the Maven repository didn't let you delete older versions. Not sure if the existence of this page means it should still be available

@codecov
Copy link

codecov bot commented Oct 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.06%. Comparing base (a8f837f) to head (7999281).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1514      +/-   ##
============================================
+ Coverage     73.02%   73.06%   +0.04%     
- Complexity     4381     4382       +1     
============================================
  Files           475      475              
  Lines         14222    14222              
  Branches       1447     1447              
============================================
+ Hits          10385    10391       +6     
+ Misses         2980     2973       -7     
- Partials        857      858       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@WilliamBergamin WilliamBergamin merged commit 1595819 into main Oct 8, 2025
6 of 7 checks passed
@WilliamBergamin WilliamBergamin deleted the update-snapshot-release-process branch October 8, 2025 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement M-T: A feature request for new functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants