thruster-rs / Thruster

A fast, middleware based, web framework written in Rust
MIT License
1.07k stars 47 forks source link

README mistake #96

Closed davedbase closed 5 years ago

davedbase commented 5 years ago

In the basic example of the README the endpoint declaration is currently:

app.get("/plaintext", middleware![plaintext]);

But the correct way seems to be:

app.get("/plaintext", middleware![Ctx => plaintext]);

Might want to adjust it, through me off at first might do the same to others.

trezm commented 5 years ago

Ah, you're absolutely right!!! Thank you so much for pointing that out! I'll get on the fix tonight when I'm near a computer

On Sun, Feb 17, 2019, 12:59 PM David Di Biase notifications@github.com wrote:

In the basic example of the README the endpoint declaration is currently:

app.get("/plaintext", middleware![plaintext]);

But the correct way seems to be:

app.get("/plaintext", middleware![Ctx => plaintext]);

Might want to adjust it, through me off at first might do the same to others.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/trezm/Thruster/issues/96, or mute the thread https://github.com/notifications/unsubscribe-auth/ABNnLTl4Fy5yCyVmVjKfFQMsfDU1-kfAks5vOZidgaJpZM4a_sdC .