<@U01J90KBSU9> I'm done pushing the mentioned chan...
# hacktoberfest-2022
c
@Samhita Alla I'm done pushing the mentioned changes except for linting issues. I was unable to sign off properly because I messed up my pre-commit hooks, will GA run?
@Eduardo Apolinario (eapolinario) Can my GA builds be run if the DCO check failed? I've to fix my git before rebasing my commits
k
yes, it can. could you share the link of the pr?
e
@Kevin Su, https://github.com/flyteorg/flytekit/pull/1243 is the PR. I approved the tests.
c
@Kevin Su I've given a go at fixing my linting errors, can the GA builds be re-run?
@Eduardo Apolinario (eapolinario) Please approve my tests, i've fixed linting errors
@Ketan (kumare3) Can you please approve my tests?
I've messed up my previous PR, while trying to fix an unexpected dep issue with flyteidl, i've opened a fresh one #1269 (https://github.com/flyteorg/flytekit/pull/1269/) Please allow tests to run @Eduardo Apolinario (eapolinario) (Sorry for the multiple pings, i'm trying to fix issues ASAP)
@Kevin Su Sorry for bothering again, can my tests be approved? Yesterday i'd to face an unexpected dependency issue with flyteidl versioning (#2522), i'd messed up my previous PR so i've opened a new PR here with all the changes (https://github.com/flyteorg/flytekit/pull/1269/)
s
@cryptic, I’ll review your changes. Give me some time.
104 Views