amazon-ion / ion-schema-rust

Rust implementation of Ion Schema
https://amazon-ion.github.io/ion-schema/sandbox
Apache License 2.0
12 stars 6 forks source link

Make ISL model constructor method naming consistent #197

Open desaikd opened 1 year ago

desaikd commented 1 year ago

All the ISL model constructor methods has names as per usage. For example to construct IslConstraint::ByteLength() constraint we have a method called byte_length, this follows the pattern defined here. This differs from the naming convention we have for Range with https://github.com/amazon-ion/ion-schema-rust/pull/190 which will have new_single_value and new_inclusive (new_*) methods. all the constructor methods for ISL model should follow the same naming convention for consistent APIs.