Skip to content

fix(cicd): yaml misconfiguration in github action#8

Merged
gforsyth merged 1 commit intosubstrait-io:mainfrom
danepitkin:danepitkin/fix-cicd
May 30, 2023
Merged

fix(cicd): yaml misconfiguration in github action#8
gforsyth merged 1 commit intosubstrait-io:mainfrom
danepitkin:danepitkin/fix-cicd

Conversation

@danepitkin
Copy link
Copy Markdown
Contributor

@danepitkin danepitkin commented May 30, 2023

No description provided.

@danepitkin danepitkin mentioned this pull request May 30, 2023
Merged
@danepitkin
Copy link
Copy Markdown
Contributor Author

danepitkin commented May 30, 2023

Tested locally with act --container-architecture linux/amd64 --artifact-server-path /tmp/artifacts and no longer receive a yaml config error. I must've forgot to test my final commit on https://github.com/substrait-io/substrait-python/pull/5/files.

gforsyth
gforsyth approved these changes May 30, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@gforsyth gforsyth left a comment

Choose a reason for hiding this comment

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

Nice! :shipit:

@gforsyth gforsyth merged commit 2e6f6a5 into substrait-io:main May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gforsyth gforsyth gforsyth approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@danepitkin @gforsyth