Skip to content
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

Методы в Configurer [WIP] #184

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Conversation

kverty
Copy link
Contributor

@kverty kverty commented Jul 20, 2016

добавлены два метода
один для генерации нового файла конфигурации модели
другой для изменения параметра модели

kverty added 5 commits July 20, 2016 22:23
один для генерации нового файла конфигурации модели
другой для изменения параметра модели
плюс возвращение методов
@codecov-io
Copy link

codecov-io commented Jul 20, 2016

Current coverage is 33.59% (diff: 16.66%)

Merging #184 into master will decrease coverage by 0.29%

@@             master       #184   diff @@
==========================================
  Files           165        165          
  Lines          4084       4147    +63   
  Methods         784        786     +2   
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           1384       1393     +9   
- Misses         2700       2754    +54   
  Partials          0          0          

Powered by Codecov. Last update 11d7da0...26404f4

{
QString content = "";
QString indent = " ";
content += "<config version = \"" + version() + "\">" + '\n';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

лучше сразу QString content = ...

@iakov
Copy link
Member

iakov commented Aug 23, 2016

@yurii-litvinov Review, please.
@kverty Add tests, please.

@iakov iakov force-pushed the master branch 16 times, most recently from 521d224 to df70e74 Compare November 29, 2018 16:17
@iakov iakov force-pushed the master branch 2 times, most recently from 977fb9a to cd7baa8 Compare January 12, 2019 14:01
@iakov iakov changed the title Методы в Configurer Методы в Configurer [WIP] Apr 14, 2020
@iakov iakov force-pushed the master branch 2 times, most recently from d1d317e to 0a4ad77 Compare November 19, 2023 17:26
@iakov iakov force-pushed the master branch 3 times, most recently from b3c87b7 to 165068c Compare August 12, 2024 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants