Giter VIP home page Giter VIP logo

Comments (3)

DavidLambauer avatar DavidLambauer commented on August 18, 2024

Hi @eadesignro,

thanks for your recommendation, I will have a look on these (hopefully tonight) and come back with feedback.

from awesome-magento2.

DavidLambauer avatar DavidLambauer commented on August 18, 2024

Hi @eadesignro, I finally came to a small review. As you can see in the contribution Guide, I would like to see some minor changes. Most things are common and not a big deal to "fix" or "improve". It just makes sure, that everybody knows the purpose of the extension and knows how to install it.

Due to that, I created a small PR (EaDesgin/magento2-pdf-generator2#3) where I added three new sections to the Readme.

A battle proofed and standard way to make things more clear is to use badges. You might want to checkout http://shields.io/ to generate some for your Repositories.

The contribution guide also points to a point, where code should be tested. I do not force contributors to add tests, but I would love to see the major parts of the extension tested.

Please check out the Contribution Guide.

This is, what I would say, a best practice extension for the M2 Awesome List. You might want to check it out:

In Summary:

  • Readme Improvements (Badges can help)
  • Add some Unit or Integration Tests to make sure, the core of the Functionality works well. If this would be done later, it's ok. I will do my best to also add a few PR's to your Extensions!

Thanks for your recommendations.

PS: I love your Videos! This is what makes Magento 2 Awesome!

from awesome-magento2.

eadesignro avatar eadesignro commented on August 18, 2024

Hi,

Thanks for the effort and time spent. We can close the issue now.

P.S. During the spring we are swamped with work but unit testing is must!

from awesome-magento2.

Related Issues (18)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.