Skip to content

Conversation

@vicheey
Copy link
Contributor

@vicheey vicheey commented Dec 6, 2025

Issue #, if available:
Recent update of setup-ruby@v1 implies update bundler to v4.0.0 which know to stop supporting --without falg used in our test:

>               raise WorkflowFailedError(workflow_name=self.NAME, action_name=action.NAME, reason=str(ex))
E               aws_lambda_builders.exceptions.WorkflowFailedError: RubyBundlerBuilder:RubyBundle - Bundler Failed: The `--without` flag has been removed because it relied on being remembered

E               across bundler invocations, which bundler no longer does. Instead please use

E               `bundle config set without 'development test'`, and stop using this flag

Description of changes

Pins the version of ruby GitHtub action, to before they updated to use bundler 4.0.0
diff: ruby/setup-ruby@v1.268.0...v1.269.0

Similar to: fix: pin ruby action to specific version by valerena · Pull Request #8491 · aws/aws-sam-cli

Description of how you validated changes

Checklist

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@vicheey vicheey requested a review from a team as a code owner December 6, 2025 00:04
@vicheey vicheey enabled auto-merge December 6, 2025 00:14
@valerena valerena closed this Dec 8, 2025
auto-merge was automatically disabled December 8, 2025 18:45

Pull request was closed

@valerena valerena reopened this Dec 8, 2025
@valerena valerena changed the title fix(dependency) pin git action setup-ruby to v1.268.0 until Lambda Builders support bundler v4.0.0 fix(dependency) pin git action setup-ruby to v1.268.0 until supporting bundler v4.0.0 Dec 8, 2025
@valerena valerena added this pull request to the merge queue Dec 8, 2025
Merged via the queue into develop with commit 719e7e9 Dec 8, 2025
96 checks passed
reedham-aws added a commit to reedham-aws/aws-lambda-builders that referenced this pull request Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants