ZutJoe / KoalaHackerNews

Koala hacker news 周报内容 每周二0点左右更新
https://zutjoe.github.io/KoalaHackerNews/
MIT License
424 stars 15 forks source link

README add video title and Adjust the compatibility of pytho versions to fit python on 3.8 to 3.9. #7

Closed tower-town closed 1 year ago

tower-town commented 1 year ago

为README添加一定量的视频标题。考虑到PR的用户在各个发行的python3默认版本(python3.8-python3.10),所以语法上使之能兼容python3.8-python3.9

ZutJoe commented 1 year ago

很感谢您的 PR, 但是您的 PR 我目前不能全部合并:

  1. 我们这个项目的定位是检索 Koala 视频中推荐的内容,因此将视频中的标题直接搬过来我认为是没必要的,但是或许添加时间是一个比较好的方法,参考 #3 ,但我还是觉得只是添加时间会没有那么美观(可能是我的审美有问题)
  2. 下面是您的 README 的图片,以我的审美来看,那还是不是很好看的,视频标题太长了。 image
  3. 兼容3.8-3.9,我认为是值得考虑的一件事。项目创建不久就出了3.11版本,我们这个项目本身的体谅不大,想要改一些东西很快,我认为尝试的去使用一些新鲜的,方便的语法会有助于我们快速掌握新的语法(我们这个项目本身就可以作为python练手项目了解)。如果可以,我认为您的代码可以提交到一个新的分支,如果您认为可以的话,我可以新开一个新的分支,作为3.8-3.9的语法版本。
  4. 使用poetry管理项目我认为可以,谢谢您的提供。我现在去学习一下。

综上,如果您愿意的话,我希望您能提两个 PR,您的兼容代码我认为可以放在另一个分支,使用poetry管理项目可以提交到主分支

tower-town commented 1 year ago

我觉得README的形式可以参考 https://github.com/wsdjeg/Learn-Vim_zh_cn 如以下:

目录

2022.11.30-Hack-News周报

ZutJoe commented 1 year ago

加目录吗,可是如果知道都是哪一期视频了为啥不直接去看视频和下面的链接,为啥要来看这个

tower-town commented 1 year ago

我觉得没有必要重新开一个分支去兼容,这完全是你本人的项目的所决定

尝试的去使用一些新鲜的,方便的语法会有助于我们快速掌握新的语法

如果为以上目的,而不考虑更多的人加入PR,,确实是应该这样。 ps: 一些信息 Debian 11 : python3.9 Alpine 3.16.2: python3.10 Anaconda: python3.9 [2022.11.30]

tower-town commented 1 year ago

目录是为了当文档变得庞大时便于管理(比如现在是48个,以后会更多),而且也不影响阅读体验。如果你觉得一期又太过零碎,可以一个月作为一个章节

ZutJoe commented 1 year ago

目录是为了当文档变得庞大时便于管理(比如现在是48个,以后会更多),而且也不影响阅读体验。如果你觉得一期又太过零碎,可以一个月作为一个章节

可以的

ZutJoe commented 1 year ago

如果为以上目的,而不考虑更多的人加入PR,,确实是应该这样。

想加入PR可以自己通过虚拟环境来使用3.10或者更新本身的python版本,我不认为一味地使用主流版本而不尝试去使用新版本是好的,那只是为了让自己舒服。新的版本有更方便的语法糖为什么不去用,反而去使用冗余的代码来代替新的方便的语法。

这个项目本身是不大的,修改起来也是很简单,如果之后的版本有新的更方便的写法,我同样会改掉,而把这个版本转为旧版本