peixubin

Results 4 comments of peixubin

"Codecov token not found. Please provide Codecov token with -t flag." 是什么意思?是不是构建脚本有问题

> 这一版还是和自动通过结合了, 在这一步之前再做一个处理, 如果 workflow status 是 autoreview_wrong ,那么就提前保存, return , 你看能不能解决你的问题. > > https://github.com/hhyo/Archery/blob/52ce7595d4b3b6e294b49c4d9a4ce10a0992d60f/sql/utils/workflow_audit.py#L303-L316 对于自动驳回和自动通过是否结合,我不能理解。驳回或通过本来就是审核的两个动作,在create_audit里面必定会存在驳回或通过两个场景,这有什么值得疑问的? 大部分代码是原来就有的,我还不想去大动。我只是增加了对工单状态的判断。 ` 239 def generate_audit_setting(self) -> AuditSetting: 240 if self.is_auto_review(): 241 if self.workflow_type in...

> 工单执行任务配置的是永不超时,执行动作本身不是幂等的也不应该有重试机制,建议通过任务下发时配置禁止重试https://django-q.readthedocs.io/en/latest/configure.html#ack-failures 重试是 django-q的默认行为,没发现可禁用的配置,只是想通过其他方法解决。 如果使用ack-failures来解决,似乎应该将该配置默认启用,类似于最佳实践

再就是“工单状态不正确,禁止执行!”的报错,对于用户来说比较模糊,看不懂。实际上在超时的情况下,任务是还在后台执行。