-
-
Notifications
You must be signed in to change notification settings - Fork 561
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
TypeError: 'NoneType' object is not subscriptable sys:1: RuntimeWarning: coroutine 'AsyncClient.get' was never awaited #463
Comments
@Snailedlt How did you fix that, as I am also getting the same error again and again? |
I just re-ran the job and it worked fine |
But in mine, it fails every time with the same error. |
@AyushAgnihotri2025 Try re-running it with debug enabled and post the complete log here |
This same issue was mentioned and being discussed on this issue. What happens: I posted my debug log there, could be the same issue for you. |
|
As the @AyushAgnihotri2025 logs says, it's trying to get commit data from one of your 127 repos, and it didn't threat the exception on not receiving it. Same issue as #425. |
Still happens for me even when using version from this PR |
@BANanaD3V Any logs? And how did you test the PR? |
Yeah, i forked it and added some prints etc etc and it failed on nixpkgs repo, when i added try except (bad thing ik) and after that it says |
any updates? |
None. I'm using this one for now, you can see it on my profile. One thing we could do is see why the this one works and this one does not, but it might need some long debugging and code changes so it does not exceed those limits. I think it might have something related to the commit count of a repo, based on the previous log files and the fact that the athul's repo doesn't load commit data, only wakatime data (I think so? Might be wrong). Would love to to these tasks but I really can't right now. |
Hey :) |
@cdfmlr tried the repo, and sadly it didn't work for me:
Same error message as always. |
@willnaoosmith It's likely to be another issue. Seem to be a network problem (502 Bad Gateway).. |
@willnaoosmith it's the same for everyone (Maybe, I hope so). See @AyushAgnihotri2025 logs in this issue, and it's the same. The code tries to fetch commit data from a big repository, and it hits an API rate on GitHub, as it looks like. [EDIT] Will wait for someone to post some logs. |
The doc say exceeding the rate limit response 200 or 403. |
@cdfmlr well pointed. Forked the repository, will try to debug it later. |
Related code: waka-readme-stats/sources/manager_download.py Lines 213 to 231 in f8cec5f
There are already retries on |
Error: - RuntimeWarning: coroutine 'AsyncClient.get' was never awaited Solution: - Uses `cdfmlr/waka-readme-stats@master` instead of `anmol098/waka-readme-stats@master` References: - https://github.com/beatrizsmerino/beatrizsmerino/actions/runs/6918817111/job/18821477514 - anmol098/waka-readme-stats#463 (comment)
Hello!! https://github.com/beatrizsmerino/beatrizsmerino/actions/runs/6918946818/job/18821734503 |
@beatrizsmerino, the issue arises because the Wakatime API responded with a status code of Source code here: waka-readme-stats/sources/manager_download.py Lines 187 to 193 in f8cec5f
However, I noticed that there is So A quick fix could be allowing the use of |
Maybe, but how can I do it? |
You can fork my branch: https://github.com/cdfmlr/waka-readme-stats/tree/master (I have made some changes in
If it works, you can make a PR to current repo (anmol098/waka-readme-stats). P.s. the last commit in this repo (anmol098/waka-readme-stats) was 8 months ago. I'd be glad to maintain my fork (cdfmlr/waka-readme-stats) temporarily until the maintainers return. Feel free to submit a pull request to my repo as well. |
Ok thank you very much for the help, I will try it right now. |
@beatrizsmerino Unfortunately, I can do nothing to your PR. I have no access to maintain this repository. |
I did something wrong because it didn't work https://github.com/beatrizsmerino/beatrizsmerino/actions/runs/6941214625/job/18881598413 |
I have not worked with python before is the condition correct?
I have not worked with python before is the condition correct? |
Sorry, I had made the fork of the other repository. |
The point is that the change of code won't make different until the author rebuild and push new the docker image. To make the code modifications take effect immediately without compiling and pushing images in the GitHub Action, you can fork my branch, as I mentioned:
Or, you may want to do this: d6f3a51 |
Ok so I just need to do the fork of
I don't need to make any changes to the repository? |
@beatrizsmerino merged into https://github.com/cdfmlr/waka-readme-stats. Please try it by:
Note: Your issue is duplicated with #482. And your fix is equal to #483. |
ok I will try again |
Excuse my delayed response. #483 is merged. Please try to switch back to |
Thank you very much for your help!!!!! |
My waka-readme job failed today due to a TypeError. After re-running it worked fine, but maybe there is an underlying issue here so I'll post the issue anyways :)
Action run: https://github.com/Snailedlt/Snailedlt/actions/runs/5422855494/jobs/9860032571
Error log:
The text was updated successfully, but these errors were encountered: