Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
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
Add CI jobs #10
Add CI jobs #10
Changes from all commits
76d4540
71f31dc
cadc4bc
3f46b26
2dbd68f
be8c202
695a133
cea436a
bc7116f
351fd7e
b18b582
c3ebe4c
cb21a8e
7e67976
487341d
e58540a
d40ce2e
cefb8f6
29b319d
b4ef9f6
a3fef15
5a36b01
37ce733
cca7db5
abeae67
8af9c15
510db4a
1335b56
cb57054
25bd2b7
1669c92
42b0c5b
40b8d91
f84ba0b
946509c
a6e76c7
f59a41a
306c1b9
7ea3af1
1b2ce4d
02906bb
91ac502
08842b1
1dc9460
a70ea01
a635037
8dcceef
5690d0a
667691f
e253487
b0b0410
e0f71aa
ab57c23
e2a2a3b
8c318ff
8dfd9b9
d1eac13
78566e0
effab51
b80ad0a
ee7841d
774b2c8
7449fcb
e11cc48
7ef7ce7
ce3bb98
6497681
f340a61
4614362
6baf691
b4db71a
ca82b03
5ea590e
110af93
c9f57b4
0789176
f016805
1f647ba
78b0b41
7e48e2b
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this needed? I think we're probably missing some other piece of setup like https://github.com/rapidsai/shared-workflows/blob/9638706c8c3e43bb0b210528a814a2abb2f3a923/.github/workflows/wheels-build.yaml#L126-L130
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Without this, I get
during the wheel build step, this is a workaround. Still following up on this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the follow-up task here is to try using the
aws-actions/configure-aws-credentials@v3
step that I linked above. It doesn't matter for this PR, though.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: not sure why we need to skip this in CI but it works for me locally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe this should be xfailed in CI instead of skipped?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we still want to skip things due to #3