Skip to content
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

修复iqqtv中文,回退并修复google翻译 #251

Merged
merged 11 commits into from
Sep 24, 2024
Merged

Conversation

anyingxiuluo
Copy link
Contributor

@anyingxiuluo anyingxiuluo commented Aug 31, 2024

  1. 修复iqqtv中文iqqtv无法正常刮削中文标题 #246
  2. 回退fix: google translate 混合多行文本这里修改的google翻译,并修复了bug,解决了因为该修改引入的补全演员信息时调用google翻译报错 #250
  3. 修复airav_cc无码破解的判断
  4. 很少用GIT提交代码,下面有很多不必要的提交不知道怎么删,望见谅

@kikiyou18
Copy link
Contributor

不要紧,作者使用的是压缩合并,无论你有多少个提交合并到主分支时都只会产生一个提交,你自己觉得没什么就行。

你也可以在自己的分支上使用 git rebase -i HEAD~4 进入交互模式,进行压缩或删除提交,或者 git cherry-pick 也可以做到合并提交,然后再拉取合并请求。怕弄乱你也可以先在本地分支试验一下,没问题再推到远程仓库,有问题直接删除仓库重新克隆就行。

@sqzw-x sqzw-x self-requested a review September 7, 2024 02:13
outline = ''.join(result[i:])
return title.strip(), outline.strip(), ''
except Exception as e:
return title, outline, f'返回数据格式异常!返回内容:{response} 错误:{e}'
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

原来的这种写法可读性为 0, 不要回退. 加一个异常处理即可

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

原来的写法标题和内容一起翻译,能减少一次google翻译请求,防止被封

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Google translate 频率限制很宽松, 没有必要多此一举

@sqzw-x sqzw-x changed the base branch from master to dev September 24, 2024 14:36
@sqzw-x sqzw-x merged commit 9543873 into sqzw-x:dev Sep 24, 2024
sqzw-x added a commit that referenced this pull request Sep 24, 2024
Co-authored-by: anyingxiuluo <125189022+anyingxiuluo@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants