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

Добавляет возможность управлять тегами barcode #11

Merged
merged 3 commits into from
Aug 28, 2018

Conversation

mmjurov
Copy link
Contributor

@mmjurov mmjurov commented Aug 28, 2018

Добавлено поле barcodes для \Bukashk0zzz\YmlGenerator\Model\Offer\AbstractOffer, а также сопутствующие методы для него.

Попутно добавил в require-dev зависимость от ext-dom, т.к. без него проверка dtd работать не будет. PhpStorm все равно ругается, просит добавить в require секцию, но я не стал.

Добавил также в \Bukashk0zzz\YmlGenerator\Tests\AbstractGeneratorTest::createOffers новое поле, генерируется с помощью Faker'а

Adding ext-dom dependency for dev environments, because tests are depends on it
@Bukashk0zzz
Copy link
Owner

Поправьте пожалуйста код стайл и я солью.

@mmjurov
Copy link
Contributor Author

mmjurov commented Aug 28, 2018

done

@codecov-io
Copy link

codecov-io commented Aug 28, 2018

Codecov Report

Merging #11 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##             master    #11   +/-   ##
=======================================
  Coverage       100%   100%           
- Complexity      238    241    +3     
=======================================
  Files            17     17           
  Lines           661    670    +9     
=======================================
+ Hits            661    670    +9
Impacted Files Coverage Δ Complexity Δ
src/Model/Offer/AbstractOffer.php 100% <100%> (ø) 48 <3> (+3) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d4cbf6...9e6541e. Read the comment docs.

@Bukashk0zzz Bukashk0zzz merged commit 62bf705 into Bukashk0zzz:master Aug 28, 2018
@mmjurov mmjurov deleted the barcode branch August 28, 2018 11:57
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.

None yet

3 participants