Remove Canapi #540

Samengevoegd

elegaanz heeft 4 commits samengevoegd van bye-canapi naar master

elegaanz reageerde (Gemigreerd van 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] reageerde (Gemigreerd van 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 beoordeeld

trinity-1686a heeft een reactie achtergelaten

Eigenaar

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

plume-api/src/posts.rs

trinity-1686a reageerde

Eigenaar

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 beoordeeld

trinity-1686a heeft een reactie achtergelaten

Eigenaar

👍

:+1:

trinity-1686a heeft deze veranderingen goedgekeurd

trinity-1686a heeft een reactie achtergelaten

Eigenaar

👍

:+1:

Reviewers

trinity-1686a heeft deze veranderingen goedgekeurd

De pull request is samengevoegd als ec57f1e687.

Je kunt ook command line instructies bekijken.

Stap 1:

Vanuit het project, check een branch uit en test de veranderingen.

git checkout -b bye-canapi master

git pull origin bye-canapi

Stap 2:

Voeg de wijzigingen samen en update ze op Forgejo.

git checkout master

git merge --no-ff bye-canapi

git push origin master

Geen beoordelaars

Geen mijlpaal

Niet toegewezen

2 deelnemers

Notificaties

Vervaldatum

De deadline is ongeldig of buiten bereik. Gebruik het formaat 'jjjj-mm-dd'.

Geen vervaldatum ingesteld.

Afhankelijkheden

Geen afhankelijkheden ingesteld.

Laden…

Er is nog geen inhoud.

Labels Mijlpalen

Remove Canapi #540

Codecov Report

Reviewers

Stap 1:

Stap 2: