Adding to mhwd the version option #87
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "unknown repository"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Created by: filipe-marques
Hi!
I add to mhwd the version option, in the menu the options -v or --version, and created the necessary member functions.
I also add a member function to set the version and copyright more easily in main.cpp, that then will
show up, if the user request to show version. With the version and copyright also the user see what is the license.
Added to source files my name and email and also in authors file.
In todo file, could someone explain in more detail, what is to do? Because the list is unconcise.
Thanks!
Created by: philmmanjaro
So far so good. Can you fix the intentions as they are way of?
Created by: philmmanjaro
TODO is outdated. @december0123, can you update it if needed?
Created by: Kirek
Can you change functions and vars to CamelCase?
For example:
void Mhwd::set_version_mhwd(std::string version_of_software, std::string year_copyright)
to
void Mhwd::setVersionMhwd(std::string versionOfSoftware, std::string yearCopyright)
Created by: philmmanjaro
Right, I forgot about that on my review.
Created by: december0123
Thank you for your contribution. As Kirek stated, please remember to use the style that you see in the code.
Created by: filipe-marques
Ok, thanks for the recommendations, I will fix that.
Created by: filipe-marques
Do I need to close this pull request ?
Created by: Kirek
I think the commits done in your mhwd fork will automatically show up in the pull request, so you don't really need to.
Created by: filipe-marques
I made the changes to Camel Case convention, check the last commit.
Created by: december0123
Ok, I'll merge it now. About TODO, I'll look into it.