projectfluent / fluent-rs

Rust implementation of Project Fluent
https://projectfluent.org
Apache License 2.0
1.07k stars 96 forks source link

Rename Message to FluentMessage #128

Closed zbraniecki closed 5 years ago

zbraniecki commented 5 years ago

how about FluentMessage, then? I'm concerned about the naming conflict with ast::Message. I understand that it's not really a problem in code, because the module namespacing works to our advantage. But conceptually, I'd prefer for these two message to have distinct names.

Originally posted by @stasm in https://github.com/projectfluent/fluent-rs/pull/120

zbraniecki commented 5 years ago

Fixed in https://github.com/projectfluent/fluent-rs/commit/b69ef7d65317fb8468812ac7edb8ea091ddbbdd3