APIFlask 开源松 Sprint 7 任务

开源松
,
1

APIFlask 任务：
Repo: APIFlask (github)

  1. 修复现在的 unit test，使其能跑起来 · APIFlask (github)
  2. Add return type information from APIFlask route method helpers · issue #666 · APIFlask (github)
  3. Add OpenTelemetry usage docs · #672 · APIFlask (github)
2
  1. 认领 Pull Request
2 个赞
3

谢谢，PR 已合并。

1 个赞
4

这个PR的处理有一些问题

我在处理自己PR的时候发现，造成这个问题的原因不是Python 3.8/3.9之间的版本差异。
而是apispec这个依赖的6.8.3版本更新造成的。

image
image2187×1319 288 KB

我已经在这条commit里处理了。

感谢您的贡献。

3 个赞
5

这个 commit 拆分出来比较好

6

:heart_eyes: Cool，谢谢，我的方案是对着测试结果来的临时绕过去的方案，还是你的彻底。

7

OK，我等会拆分出来

8

这个 commit 我单独 cherry-pick 了一个PR

3 个赞
9

@uncle-lv @greyli 我们是否有需要引入 DCO 让项目更规范点（看起来）？

10

感觉意义不大，先专注功能开发吧。这个以后再说

1 个赞
11
  1. add otel example in #677
1 个赞
12

第二个任务简易的添加了 type，但是感觉这么处理有点奇怪，有没有更好的建议？@greyli @uncle-lv

PR：Add return type information from APIFlask route method helpers by FarmerChillax · Pull Request #678 · apiflask/apiflask · GitHub

13

我对照着AI的建议也做了一版Task 2，一会也提交一下

14

考虑到整的库的兼容性还有测试用例，沿用了auth_required的返回值定义 PR: Tyang ck sprint7 task2 by weidesh · Pull Request #681 · apiflask/apiflask · GitHub

2 个赞
15

忘记 squash 了……下次记得多用 rebase 来保持 commit 历史干净哦

16

沿用 Flask 的类型用法问题不大

17

我更偏向 #678 的处理，和 Flask 官方的定义保持一致。感谢贡献！

1 个赞
18

用习惯了 gitlab 的自动 squash 忘记做 rebase 了 :joy:

引申话题：你觉得添加一个打 label 自动做 auto squash 的机器人如何？issue：#683
如果觉得还不错的话我研究一下怎么添加这个功能

19

没事，暂时不用

1 个赞