<@U01J90KBSU9> I'm done pushing the mentioned chan...
# hacktoberfest-2022
b
@tall-lock-23197 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?
@high-accountant-32689 Can my GA builds be run if the DCO check failed? I've to fix my git before rebasing my commits
g
yes, it can. could you share the link of the pr?
h
@glamorous-carpet-83516, https://github.com/flyteorg/flytekit/pull/1243 is the PR. I approved the tests.
b
@glamorous-carpet-83516 I've given a go at fixing my linting errors, can the GA builds be re-run?
@high-accountant-32689 Please approve my tests, i've fixed linting errors
@freezing-airport-6809 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 @high-accountant-32689 (Sorry for the multiple pings, i'm trying to fix issues ASAP)
@glamorous-carpet-83516 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/)
t
@billowy-continent-33554, I’ll review your changes. Give me some time.
155 Views