Yuekui

Results 10 comments of Yuekui

Instead of using `pull_request_target`, we could check repo's owner to skip lint action: ``` - name: Run Lint uses: wearerequired/lint-action@v1 if: ${{ github.event_name != 'pull_request' || github.event.pull_request.head.repo.owner.login == github.repository_owner }}...

> > Hey good people, I'm a new maintainer here. you can ping me anytime for reviewing and insights for direction. if we are unsure about anything we then ping...

Hi @auvipy please let me know if there's any improvement we could do for this PR.

> we got a regression report, also a possible fix, can you check? https://github.com/encode/django-rest-framework/pull/9314 Yeah that should fix the case that if only get_object() was defined, it looks good to...

This repo is no longer active, try [this](https://github.com/jd/tenacity#stopping)

Potential fix: https://github.com/encode/django-rest-framework/pull/8082

We've already setup comment without `feedback`: ![image](https://github.com/codecov/feedback/assets/2561450/7d2663a9-bdc1-44fd-b984-0cffa26850a9) But why we're still getting this `feedback` section? ![image](https://github.com/codecov/feedback/assets/2561450/f42d51bc-d886-41ab-a9ae-18053f02f62a)

> qq @yuekui @mjbogusz : are you facing 500 error exceptions with this? In other words, are you having to manually catch this edge case? I noticed that the DRF...

This should already be fixed by https://github.com/hashicorp/terraform-provider-aws/pull/37686 in [terraform-provider-aws v5.51.1](https://github.com/hashicorp/terraform-provider-aws/releases/tag/v5.51.1)