search

ArieGato / serilog-sinks-rabbitmq

Serilog Sink for RabbitMq
Apache License 2.0
53 stars 51 forks source link

Add API approval tests #141

Closed sungam3r closed 6 months ago

sungam3r commented 6 months ago

See serilog or destructurama repos as examples.

@ArieGato I can post PR if ok.

ArieGato commented 6 months ago

What the purpose for this? Can you explain a little?

sungam3r commented 6 months ago

These tests allow to verify that public APIs were changed intentionally. It helps in versioning. It's one very small test + txt file representing snapshot of the current API set of tested assembly. Many repos use such approach.

ArieGato commented 6 months ago

So basically it's to detect breaking changes? Sounds like a good plan then.

sungam3r commented 6 months ago

So basically it's to detect breaking changes?

Yes.