Skip to content

fix(request): anticipate operations with no inputs - #108

Merged
Tieske merged 1 commit into
mainfrom
fix/params
Mar 19, 2024
Merged

fix(request): anticipate operations with no inputs#108
Tieske merged 1 commit into
mainfrom
fix/params

Conversation

@Tieske

@Tieske Tieske commented Mar 19, 2024

Copy link
Copy Markdown
Contributor

If no inputs were available, this would throw hard errors.

@github-actions

github-actions Bot commented Mar 19, 2024

Copy link
Copy Markdown

Luacheck Report

3 tests   - 36   0 ✅ ± 0   0s ⏱️ ±0s
1 suites ± 0   0 💤 ± 0 
1 files   ± 0   3 ❌  - 36 

For more details on these failures, see this check.

Results for commit a47ea82. ± Comparison against base commit f6a4468.

This pull request removes 39 and adds 3 tests. Note that renamed tests count towards both.
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:1
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:10
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:11
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:12
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:13
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:14
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:15
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:16
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:17
spec/03-credentials/04-RemoteCredentials_spec.lua ‑ spec/03-credentials/04-RemoteCredentials_spec.lua:18
…
src/resty/aws/init.lua ‑ src/resty/aws/init.lua:1
src/resty/aws/request/build.lua ‑ src/resty/aws/request/build.lua:1
src/resty/aws/request/build.lua ‑ src/resty/aws/request/build.lua:2

♻️ This comment has been updated with latest results.

@Tieske
Tieske merged commit 9c7c84c into main Mar 19, 2024
@Tieske
Tieske deleted the fix/params branch March 19, 2024 19:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants