Repository navigation
Add __main__.__cached__ - #6276
Conversation
|
Warning Rate limit exceeded@youknowone has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 14 minutes and 46 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (4)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
| } | ||
|
|
||
| fn run_any_file(&self, scope: Scope, path: &str) -> PyResult<()> { | ||
| let path = if path.is_empty() { "???" } else { path }; |
There was a problem hiding this comment.
Are those ??? is something that CPython is also doing?
There was a problem hiding this comment.
Yes, _PyRun_AnyFileObject does
| if pyc { | ||
| todo!("running pyc is not implemented yet"); | ||
| } else { | ||
| if path != "<stdin>" { |
There was a problem hiding this comment.
How feasible doing something like:
enum InputFile {
StdIn,
Path(&str),
Pyc(&str)
}be?
There was a problem hiding this comment.
Thanks, that sounds like a good suggestion.
Currently, I am not sure how stdin can come here in RustPython. I ported a part of CLI run, but not entirely.
I will leave a comment about this to make sure it can be checked later.
de0de12 to
fb103f3
Compare
fb103f3 to
b8f3106
Compare
fix #6274
I also re-organize vm.run_script inner functions to prepare #6273