Cargo-edit: Estabilidade da API quando usada como biblioteca?

Criado em 6 dez. 2019  ·  5Comentários  ·  Fonte: killercup/cargo-edit

Olá, recentemente juntei um PR para cargo audit para adicionar um recurso cargo audit --fix , ala npm audit fix , que usa cargo-edit -as-a-library para executar o Atualizações:

https://github.com/RustSec/cargo-audit/issues/23

Ao fazer isso, fixei muito especificamente em = 0.4.1 para evitar quebras (especificamente, gostaria de alinhar em torno das versões git2 - veja #341), mas estava curioso para saber se isso seria possível ter um requisito de versão mais flexível.

Parece que bloquear para uma versão específica foi uma boa ideia, porque parece que já somos incompatíveis com o cargo-edit v0.4.2:

https://github.com/RustSec/cargo-audit/pull/168/files#diff -c196526203005d3b71665eeedc3fec4c

Especificamente, para LocalManifest::upgrade (que usamos aqui ):

this function takes 3 parameters but 2 parameters were supplied

Quaisquer pensamentos sobre colocar pelo menos partes da API sob a versão do Cargo em 0.x semver, e fazer alterações como esta um pequeno aumento de versão? (por exemplo, 0,4 -> 0,5)

picture tarcieri

Todos 5 comentários

@tarcieri obrigado pelo relato! Acho que devemos seguir semver para a biblioteca (inteira), pois não nos custa um braço e uma perna para bater a versão menor em caso de uma alteração importante. @killercup , por que?

ordian picture ordian em 6 dez. 2019
👍1

(tivemos uma discussão semelhante em https://github.com/killercup/cargo-edit/pull/332#issuecomment-522227192)

ordian picture ordian em 6 dez. 2019

Sim, acordado. Eu esqueço totalmente que isso não é apenas uma CLI, desculpe! Fique à vontade
para bater em 0,5 e puxar 0,4,2!

Em sex, 6 de dezembro de 2019, 17:50 Andronik Ordian, [email protected] escreveu:

(tivemos uma discussão semelhante em #332 (comentário)
https://github.com/killercup/cargo-edit/pull/332#issuecomment-522227192 )

—
Você está recebendo isso porque foi mencionado.
Responda a este e-mail diretamente, visualize-o no GitHub
https://github.com/killercup/cargo-edit/issues/366?email_source=notifications&email_token=AAAE4X6U5UWEW44Q6HOLSR3QXJ7FZA5CNFSM4JW4NEL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGEV2NI#issuecom5WW2ZLOORPWSZGOEGEV2NI
ou cancelar
https://github.com/notifications/unsubscribe-auth/AAAE4XZPSTVKFLWK6FTVPCTQXJ7FZANCNFSM4JW4NELQ
.

killercup picture killercup em 7 dez. 2019
👍1

Maravilha, obrigado!

tarcieri picture tarcieri em 7 dez. 2019

Além disso, se não for muito problema, seria ótimo se você pudesse fazer uma versão menor sempre que atualizar o git2.

Estamos pensando em usar seu recurso vendored-openssl , e seria bom poder garantir a unificação de recursos para a mesma versão. Caso contrário, é uma de nossas dependências mais dolorosas devido ao link de biblioteca externa, então é super bom não ter que ajudar as pessoas a depurar duas versões diferentes semver incompatíveis.

tarcieri picture tarcieri em 7 dez. 2019
👍1
Esta página foi útil?
0 / 5 - 0 avaliações

Questões relacionadas

Dowwie picture Dowwie  ·  5Comentários
jcgruenhage picture jcgruenhage  ·  7Comentários
killercup picture killercup  ·  14Comentários
joshtriplett picture joshtriplett  ·  8Comentários
killercup picture killercup  ·  19Comentários