nash-io / openlimits

A Rust high performance cryptocurrency trading API with support for multiple exchanges and language wrappers.
http://openlimits.io
BSD 2-Clause "Simplified" License
278 stars 44 forks source link

Add rules for merging to master #130

Closed notdanilo closed 2 years ago

notdanilo commented 3 years ago

Merging should be blocked otherwise.

notdanilo commented 3 years ago

These deny slows down development a bit. Sometimes I just want to remove something to test and I have to remove unused uses. I think it makes sense to deny(unstable_features), but it's OK to keep the other. Everyone should remember to cleanup the warning before pushing. We can even make the CI to fail if it sees warning messages.

#![deny(unstable_features)]
// #![deny(
//     unused_must_use,
//     unused_mut,
//     unused_imports,
//     unused_import_braces
// )]