Skip to content

Commit 77ef41e

Browse files
reedham-awsreedham-aws
committed
Revert "add return code warning"
This reverts commit a4e2ce7.
1 parent a4e2ce7 commit 77ef41e

File tree

3 files changed

+256
-2
lines changed

3 files changed

+256
-2
lines changed

‎.github/workflows/build.yml‎

Lines changed: 253 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,155 @@ jobs:
7979
- name: Functional Testing
8080
run: make func-test
8181

82+
node-integration:
83+
name: ${{ matrix.os }} / ${{ matrix.python }} / node ${{ matrix.nodejs }} / npm ${{ matrix.npm }}.x
84+
if: github.repository_owner == 'aws'
85+
runs-on: ${{ matrix.os }}
86+
strategy:
87+
fail-fast: false
88+
matrix:
89+
os:
90+
- ubuntu-latest
91+
- windows-latest
92+
python:
93+
- "3.13"
94+
npm:
95+
- 8
96+
- 9
97+
- 10
98+
- 11
99+
nodejs:
100+
- 20
101+
- 22
102+
- 24
103+
steps:
104+
- uses: actions/checkout@v6
105+
- uses: actions/setup-python@v6
106+
with:
107+
python-version: ${{ matrix.python }}
108+
- uses: actions/setup-node@v6
109+
with:
110+
node-version: ${{ matrix.nodejs }}
111+
- if: ${{ matrix.npm }}
112+
run: npm install -g npm@${{ matrix.npm }}
113+
- run: npm --version
114+
- run: make init
115+
- run: pytest -vv tests/integration/workflows/nodejs_npm
116+
117+
node-esbuild-integration:
118+
name: ${{ matrix.os }} / ${{ matrix.python }} / esbuild / node ${{ matrix.nodejs }} / npm ${{ matrix.npm }}.x
119+
if: github.repository_owner == 'aws'
120+
runs-on: ${{ matrix.os }}
121+
strategy:
122+
fail-fast: false
123+
matrix:
124+
os:
125+
- ubuntu-latest
126+
- windows-latest
127+
python:
128+
- "3.13"
129+
npm:
130+
- 8
131+
- 9
132+
- 10
133+
- 11
134+
nodejs:
135+
- 20
136+
- 22
137+
- 24
138+
steps:
139+
- uses: actions/checkout@v6
140+
- uses: actions/setup-python@v6
141+
with:
142+
python-version: ${{ matrix.python }}
143+
- uses: actions/setup-node@v6
144+
with:
145+
node-version: ${{ matrix.nodejs }}
146+
- if: ${{ matrix.npm }}
147+
run: npm install -g npm@${{ matrix.npm }}
148+
- run: npm --version
149+
- run: make init
150+
- run: pytest -vv tests/integration/workflows/nodejs_npm_esbuild
151+
152+
golang-integration:
153+
name: ${{ matrix.os }} / ${{ matrix.python }} / golang
154+
if: github.repository_owner == 'aws'
155+
runs-on: ${{ matrix.os }}
156+
strategy:
157+
fail-fast: false
158+
matrix:
159+
os:
160+
- ubuntu-latest
161+
- windows-latest
162+
python:
163+
- "3.13"
164+
steps:
165+
- uses: actions/checkout@v6
166+
- uses: actions/setup-python@v6
167+
with:
168+
python-version: ${{ matrix.python }}
169+
- uses: actions/setup-go@v6
170+
with:
171+
go-version: '^1.16'
172+
- run: make init
173+
- run: pytest -vv tests/integration/workflows/go_modules
174+
175+
java-maven-integration:
176+
name: ${{ matrix.os }} / ${{ matrix.python }} / java maven / java ${{ matrix.java }}
177+
if: github.repository_owner == 'aws'
178+
runs-on: ${{ matrix.os }}
179+
strategy:
180+
fail-fast: false
181+
matrix:
182+
os:
183+
- ubuntu-latest
184+
- windows-latest
185+
python:
186+
- "3.13"
187+
java:
188+
- "21"
189+
- "25"
190+
steps:
191+
- uses: actions/checkout@v6
192+
- uses: actions/setup-python@v6
193+
with:
194+
python-version: ${{ matrix.python }}
195+
- uses: actions/setup-java@v5
196+
with:
197+
distribution: 'corretto'
198+
java-version: ${{ matrix.java }}
199+
- run: make init
200+
- run: pytest -vv tests/integration/workflows/java_maven
201+
202+
java-gradle-integration:
203+
name: ${{ matrix.os }} / ${{ matrix.python }} / java gradle / java ${{ matrix.java }}
204+
if: github.repository_owner == 'aws'
205+
runs-on: ${{ matrix.os }}
206+
env:
207+
GRADLE_OPTS: -Dorg.gradle.daemon=false
208+
strategy:
209+
fail-fast: false
210+
matrix:
211+
os:
212+
- ubuntu-latest
213+
- windows-latest
214+
python:
215+
- "3.13"
216+
java:
217+
- "21"
218+
- "25"
219+
steps:
220+
- uses: actions/checkout@v6
221+
- uses: actions/setup-python@v6
222+
with:
223+
python-version: ${{ matrix.python }}
224+
- uses: actions/setup-java@v5
225+
with:
226+
distribution: 'zulu'
227+
java-version: ${{ matrix.java }}
228+
- run: make init
229+
- run: pytest -vv tests/integration/workflows/java_gradle
230+
82231
custom-make-integration:
83232
name: ${{ matrix.os }} / ${{ matrix.python }} / custom make
84233
if: github.repository_owner == 'aws'
@@ -99,6 +248,35 @@ jobs:
99248
- run: make init
100249
- run: pytest -vv tests/integration/workflows/custom_make
101250

251+
python-integration:
252+
name: ${{ matrix.os }} / ${{ matrix.python }} / python
253+
if: github.repository_owner == 'aws'
254+
runs-on: ${{ matrix.os }}
255+
strategy:
256+
fail-fast: false
257+
matrix:
258+
os:
259+
- ubuntu-latest
260+
- windows-latest
261+
python:
262+
- "3.9"
263+
- "3.10"
264+
- "3.11"
265+
- "3.12"
266+
- "3.13"
267+
- "3.14"
268+
steps:
269+
- uses: actions/checkout@v6
270+
- uses: actions/setup-python@v6
271+
with:
272+
python-version: ${{ matrix.python }}
273+
- run: |
274+
python -m pip install --upgrade pip
275+
pip install --upgrade setuptools
276+
if: ${{ matrix.os }} == 'ubuntu-latest' && ${{ matrix.python }} == '3.12'
277+
- run: make init
278+
- run: pytest -vv tests/integration/workflows/python_pip
279+
102280
ruby-integration:
103281
name: ${{ matrix.os }} / ${{ matrix.python }} / ruby
104282
if: github.repository_owner == 'aws'
@@ -121,5 +299,79 @@ jobs:
121299
ruby-version: "3.2"
122300
- run: make init
123301
- run: ruby -rbundler -e'p Bundler::CLI.printable_commands(true)'
124-
- run: bundle cli_help
302+
if: matrix.os == 'windows-latest'
303+
- run: bundle.bat cli_help
304+
if: matrix.os == 'windows-latest'
125305
- run: pytest -vv tests/integration/workflows/ruby_bundler
306+
307+
dotnet-integration:
308+
name: ${{ matrix.os }} / ${{ matrix.python }} / dotnet
309+
if: github.repository_owner == 'aws'
310+
runs-on: ${{ matrix.os }}
311+
strategy:
312+
fail-fast: false
313+
matrix:
314+
os:
315+
- ubuntu-latest
316+
- windows-latest
317+
python:
318+
- "3.13"
319+
steps:
320+
- uses: actions/checkout@v6
321+
- uses: actions/setup-python@v6
322+
with:
323+
python-version: ${{ matrix.python }}
324+
- run: make init
325+
- run: pytest -vv tests/integration/workflows/dotnet_clipackage
326+
327+
rust-cargo-lambda-integration:
328+
name: ${{ matrix.os }} / ${{ matrix.python }} / rust-cargo-lambda
329+
if: github.repository_owner == 'aws'
330+
runs-on: ${{ matrix.os }}
331+
env:
332+
CARGO_LAMBDA_VERSION: 0.15.0
333+
defaults:
334+
run:
335+
shell: bash
336+
strategy:
337+
fail-fast: false
338+
matrix:
339+
os:
340+
- ubuntu-latest
341+
- windows-latest
342+
python:
343+
- "3.13"
344+
rust:
345+
- stable
346+
steps:
347+
- uses: actions/checkout@v6
348+
- uses: actions/setup-python@v6
349+
with:
350+
python-version: ${{ matrix.python }}
351+
352+
# Install and configure Rust
353+
- name: Install rustup
354+
run: |
355+
: install rustup if needed
356+
if ! command -v rustup &> /dev/null ; then
357+
curl --proto '=https' --tlsv1.2 --retry 10 --retry-connrefused -fsSL "https://sh.rustup.rs" | sh -s -- --default-toolchain none -y
358+
echo "${CARGO_HOME:-$HOME/.cargo}/bin" >> $GITHUB_PATH
359+
fi
360+
if: ${{ matrix.os }} == 'ubuntu-latest'
361+
- name: rustup toolchain install ${{ matrix.rust }}
362+
run: rustup toolchain install ${{ matrix.rust }} --profile minimal --no-self-update
363+
- run: rustup default ${{ matrix.rust }}
364+
- run: |
365+
: disable incremental compilation
366+
echo CARGO_INCREMENTAL=0 >> $GITHUB_ENV
367+
- run: |
368+
: enable colors in Cargo output
369+
echo CARGO_TERM_COLOR=always >> $GITHUB_ENV
370+
371+
# Install and configure Cargo Lambda
372+
- name: Install Cargo Lambda
373+
run: pip install cargo-lambda==$CARGO_LAMBDA_VERSION
374+
- run: echo "$HOME/.local/bin" >> $GITHUB_PATH
375+
376+
- run: make init
377+
- run: pytest -vv tests/integration/workflows/rust_cargo

‎aws_lambda_builders/workflows/ruby_bundler/bundler.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ def __init__(self, osutils, bundler_exe=None):
3636
self.osutils = osutils
3737
if bundler_exe is None:
3838
if osutils.is_windows():
39-
bundler_exe = "bundler.bat"
39+
bundler_exe = "bundle.bat"
4040
else:
4141
bundler_exe = "bundle"
4242

‎tests/integration/workflows/ruby_bundler/test_ruby.py‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,8 @@
1919
("runtime",),
2020
[
2121
("ruby3.2",),
22+
# ("ruby3.3",),
23+
# ("ruby3.4",),
2224
],
2325
)
2426
class TestRubyWorkflow(TestCase):

0 commit comments

Comments
 (0)