-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Minor message tweaks #14
Conversation
Thanks for the fixes @dtchepak. Can you also fix visual basic resource file? And I think as you changed the messages the tests will be failing now |
@tpodolak Ah of course, sorry! (I just adjusted the messages via the GH online editor rather than doing it properly.) I'll resend the corrected PR once I get a chance. |
@tpodolak Think I've fixed my mistake now. 😅 That's a surprising amount of work! I'll never ask you for a message change again! 😂 Is it possible to have a shared dictionary for messages and have per-language dictionaries for just the things that vary? ( It may also be worth updating the tests to also use the resource dictionary entries -- I think testing that we have the intended message defined in the resources is more important than double-checking the message wording. I think a single source-of-truth for those is ok. I'm happy to try to migrate the tests to this approach if it sounds reasonable? I'm probably missing lots of the details sorry, so please point out any problems with this idea. :) |
By the way, if you're happy with this, please "Squash and merge" into the original PR, then merge |
Shared resource file sounds like a good idea, not sure if .net allows merging resx files, but even if it doesn't we can always "manualy" use the one we are interested in. When it comes to test changes I think we can do it as well, this will save time in the future for sure, although some might say that this is anti-pattern as stated in here https://enterprisecraftsmanship.com/2018/01/30/leaking-domain-knowledge-tests/. Even though feel free to change them, from my perspective advantages outweigh disadvantages in this case. |
Some very minor message tweaks. Mainly fixing spacing after a ".", but also ended up tweaking the ctor mismatch messages as well.