Cassandra McCarthy

Results 40 comments of Cassandra McCarthy

I think it would be good to just link to the docs in a follow up, yes

There could still be a future-proofing code change made here. We could make `entity_metadata: Option`, and add a check on each method that modifies it, which nulls the field itself...

Marking as draft until the gateway refactor merges

I tend to agree with this. I have one design question though: how should we report multiple errors from different parts of the builder? Say the content is too long...

> Builders in frameworks like `http` just don't report multiple errors, they just [report the first error encountered](https://docs.rs/http/0.2.5/src/http/request.rs.html#779-789). It seems we also have precedence for this in the `EmbedBuilder`. This...

I can get to this by the weekend.

Could you update this for https://github.com/discord/discord-api-docs/pull/5093? Two birds one stone, I figure it's in scope

Since the entire crate is refactored by #1809, I'll add it to that issue and close the component ones.

I like these! I must ask, why does the function wrap to two less than the reported character length?