Add Commands fields in Entrypoint

[jschwinger233]

No description provided.

[jschwinger233]

fa, core/types.Entrypoint 直接被 cli 引用, 用来 Unmarshal spec.yaml, 所以 types.Entrypoint 也要做一下字段兼容, 不然旧式的 spec 不能用 cli 解析.

旧式 cmd:

appname: 
entrypoints:
  zc:
    cmd: eru-core --config /etc/eru/core.yaml

新式 cmd:

appname:
entrypoints:
  zc:
    cmd:
    - eru-core
    - --config
    - /etc/eru/core.yaml

[CMGS]

fa, core/types.Entrypoint 直接被 cli 引用, 用来 Unmarshal spec.yaml, 所以 types.Entrypoint 也要做一下字段兼容, 不然旧式的 spec 不能用 cli 解析.

旧式 cmd:

appname: 
entrypoints:
  zc:
    cmd: eru-core --config /etc/eru/core.yaml

新式 cmd:

appname:
entrypoints:
  zc:
    cmd:
    - eru-core
    - --config
    - /etc/eru/core.yaml

cli 改一下就好了吧- -

[CMGS]

不过讲道理，也不是每个人都需要 line by line 的那还是兼容吧……你看这个例子，多丑- -

[jschwinger233]

不过讲道理，也不是每个人都需要 line by line 的那还是兼容吧……你看这个例子，多丑- -

嗯, 不过我在 cli 里做兼容吧, core/types.Entrypoint 只有一个字段 Commands []string, 不然太脏了

[CMGS]

这个我同意，对内只有一个定义，外部拆开

…

On Fri, Apr 16, 2021 at 3:46 PM zc ***@***.***> wrote: 不过讲道理，也不是每个人都需要 line by line 的那还是兼容吧……你看这个例子，多丑- - 嗯, 不过我在 cli 里做兼容吧, core/types.Entrypoint 只有一个字段 Commands []string, 不然太脏了 — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#389 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAD3S2FANQK6SDLURLW64RLTI7TMNANCNFSM43A3CTCQ> .

-- - CMGS

[tonicmuroq]

cli 那个简直是... 是有 bug 的, 现在的

[jschwinger233]

cli 那个简直是... 是有 bug 的, 现在的

你说的 bug, 和我解决的是同一个 bug 吗? https://github.com/projecteru2/cli/pull/64/files

[tonicmuroq]

cli 那个简直是... 是有 bug 的, 现在的

你说的 bug, 和我解决的是同一个 bug 吗? https://github.com/projecteru2/cli/pull/64/files

对的就是这个! 我前天改 lambda 的时候发现了...