TogetherOS / cicada

🚀 Fast lightweight HTTP service framework.
https://crossoverjie.top/categories/cicada/
Apache License 2.0
953 stars 212 forks source link

加入注解指定http配置 #50

Open tangtj1 opened 4 years ago

tangtj1 commented 4 years ago

加入@CicadaRoute Http Method 配置

使用url+http method组合为映射键

  1. 未配置默认支持POST,GET,PUT,PATCH,DELETED
  2. url+http method 组合映射的java方法不可重复
crossoverJie commented 4 years ago

不好意思,这类大的 features 通常很难 merge,原因有以下几点:

  1. 可能设计实现思路与项目本身不符。
  2. 代码太多,很难 review 及测试。

所以通常这类新的 features 一般都是 fork 之后自己维护,或者和作者沟通后今后再实现这一 features 时可以探讨和参考。

如果是一个组织的成员,大家一般都会在线下(私下沟通)达成一致之后再做开发和 PR.

基于以上几点建议大家对开源项目的 pr 可以是对 bug 的修复,或者是小部分可控代码的优化,或者分批提交,不然作者很难控制整个项目的“风格”及“方向”。