Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: check length of args of python model function before accessing i… #6049

Merged
merged 1 commit into from
Oct 11, 2022
Merged

fix: check length of args of python model function before accessing i… #6049

merged 1 commit into from
Oct 11, 2022

Conversation

ChenyuLInx
Copy link
Contributor

…t (#6042)

  • fix: check length of args of python model function before accessing it

  • Add test

  • changie

(cherry picked from commit dcd6ef7)

resolves #

Description

Checklist

@chamini2 @ChenyuLInx
fix: check length of args of python model function before accessing it (
34c2cdc 
#6042)

* fix: check length of args of python model function before accessing it

* Add test

* changie

(cherry picked from commit dcd6ef7)
@cla-bot cla-bot bot added the cla:yes label Oct 11, 2022
@ChenyuLInx ChenyuLInx marked this pull request as ready for review October 11, 2022 22:06
@ChenyuLInx ChenyuLInx requested review from a team as code owners October 11, 2022 22:06
@leahwicz leahwicz merged commit 65f40b3 into 1.3.latest Oct 11, 2022
@leahwicz leahwicz deleted the backport-6042-to-1.3.latest branch October 11, 2022 23:18
@aranke aranke mentioned this pull request Jul 19, 2023
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leahwicz leahwicz leahwicz approved these changes

@gshank gshank Awaiting requested review from gshank gshank was automatically assigned from dbt-labs/core-team

@MichelleArk MichelleArk Awaiting requested review from MichelleArk MichelleArk was automatically assigned from dbt-labs/core-group

Assignees
No one assigned
Labels
cla:yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ChenyuLInx @leahwicz @chamini2