googleapis / googleapis/google-cloud-python

Replace `except:` and `except Exception:` with a more granular exception

Đang mở
#16,201 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
priority: p2 type: process
Ngôn ngữ chính
Python
Star
5.4k
Fork
1.8k
Merge trung bình
3 ngày 4 giờ
Pull request đã merge (30 ngày)
122

Mô tả

See feedback from gemini in PR https://github.com/googleapis/gapic-generator-python/pull/2549:

https://github.com/googleapis/gapic-generator-python/pull/2549#discussion_r2747934082

```
The except: on the next line is a bare except, which is a bad practice as it catches all exceptions, including system-exiting ones like SystemExit and KeyboardInterrupt. This can make debugging difficult and hide underlying issues. It would be better to catch a more specific exception, like Exception.

While there's a TODO in the template to remove this try/except block, improving it in the meantime would be beneficial.

This change should be applied to the template file: gapic/templates/%namespace/%name_%version/%sub/services/%service/_shared_macros.j2.

Suggested change in the template:

except Exception:
```

We should remove all usages of `except:` and replace them with `except Exception:` or a more granular exception

Search results for `except:`

https://github.com/search?q=repo%3Agoogleapis%2Fgapic-generator-python%20except%3A&type=code

Search results for `except Exception:`
https://github.com/search?q=repo%3Agoogleapis%2Fgapic-generator-python+%22except+Exception%3A%22&type=code

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.