Alir3z4 / Alir3z4/html2text

proposal - default bodywidth to `None`

未关闭
#265 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Python
星标
2.2k
派生
296
PR 合并指标
30 天内没有已合并 PR

描述

`bodywidth` is currently specified in an init argument, defaulting to `config.BODY_WIDTH`.

https://github.com/Alir3z4/html2text/blob/master/html2text/__init__.py#L43

A problem with this approach: the `__init__` function creates the default value for bodywidth when it is interpreted. changing the value of `config.BODY_WIDTH`after importing html2text will have no effect.

I propose the following:

1. change the init value `bodywidth=None`
2. change the following line:

```
- self.body_width = bodywidth # covered in cli
+ self.body_width = bodywidth if bodywidth is not None else config.BODY_WIDTH # covered in cli
```

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。