Remove Canapi #540

Sloučený

elegaanz sloučil 4 commity z větve bye-canapi do větve master před

elegaanz okomentoval (Migrováno z github.com)

It added more complexity than it helped.

I will eventually rewrite it as a proc-macro that generates types for an API from an OpenAPI spec, so maybe it will be re-introduced at some point (but it will be totally different).

Also I added an endpoint to delete posts, the function was here before, but not exposed.

It added more complexity than it helped. I will eventually rewrite it as a proc-macro that generates types for an API from an OpenAPI spec, so maybe it will be re-introduced at some point (but it will be totally different). Also I added an endpoint to delete posts, the function was here before, but not exposed.

codecov[bot] okomentoval (Migrováno z github.com)

Codecov Report

Merging #540 into master will increase coverage by 0.3%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master     #540     +/-   ##
=========================================
+ Coverage   43.97%   44.28%   +0.3%     
=========================================
  Files          68       68             
  Lines        7890     7835     -55     
=========================================
  Hits         3470     3470             
+ Misses       4420     4365     -55

# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/540?src=pr&el=h1) Report > Merging [#540](https://codecov.io/gh/Plume-org/Plume/pull/540?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/a20d2df50aaf39ab7e40a76c074449fac44be617?src=pr&el=desc) will **increase** coverage by `0.3%`. > The diff coverage is `0%`. ```diff @@ Coverage Diff @@ ## master #540 +/- ## ========================================= + Coverage 43.97% 44.28% +0.3% ========================================= Files 68 68 Lines 7890 7835 -55 ========================================= Hits 3470 3470 + Misses 4420 4365 -55 ```

trinity-1686a posoudil

trinity-1686a zanechal komentář

Vlastník

small comment but not a real problem as it was already that way before

plume-api/src/posts.rs

trinity-1686a okomentoval

Vlastník

I think it should be chosen automatically only if the user has only one blog, so to be clearly deterministic (currently it's choosing the first found, which was already the old behavior)

trinity-1686a posoudil

trinity-1686a zanechal komentář

Vlastník

👍

:+1:

trinity-1686a schválil tyto změny

trinity-1686a zanechal komentář

Vlastník

👍

:+1:

Posuzovatelé

trinity-1686a schválil tyto změny

Požadavek na natažení byl sloučen jako ec57f1e687.

Můžete také zobrazit instrukce příkazové řádky.

Krok 1:

Z vašeho repositáře projektu se podívejte na novou větev a vyzkoušejte změny.

git checkout -b bye-canapi master

git pull origin bye-canapi

Krok 2:

Slučte změny a aktualizujte je na Forgejo.

git checkout master

git merge --no-ff bye-canapi

git push origin master

Přihlaste se pro zapojení do konverzace.

Žádní posuzovatelé

Bez milníku

Bez zpracovatelů

2 účastníků

Oznámení

Termín dokončení

Termín dokončení není platný nebo je mimo rozsah. Použijte prosím formát „rrrr-mm-dd“.

Žádný termín dokončení.

Závislosti

Nejsou nastaveny žádné závislosti.

Načítá se…

Není zde žádný obsah.

Štítky Milníky

Remove Canapi #540

Codecov Report

Posuzovatelé

Krok 1:

Krok 2: