[D7] Commerce Remove Cart Vat Tax

Thank you for your contribution!
7.x-1.0 is not a correct branch name. It is a correct tag name, but there cannot be a branch and a tag with the same name. The 7.x-1.0 branch needs to be removed.

Hi @bsumit5577,

First of all, the module appears to work - no configuration required as promised by the README.

You do have some results to review from pareview: https://pareview.sh/pareview/https-git.drupal.org-project-commerce_remov...

Using arg() isn't a great practice, but I also can't think of any other way to accomplish what you're trying to do, other than maybe calling menu_get_item() and checking page callbacks, but that seems unreliable.

I would also recommend adding more dependencies to your module - I found it difficult to test because I had to enable several more commerce modules, for instance, the cart module.

In your README.txt you have some unnecessary line wrapping under the "INSTALLATION" section. I would also provide a link as per https://www.drupal.org/node/2181737, maybe something like:

Install as you would normally install a contributed Drupal module. Visit:
https://www.drupal.org/documentation/install/modules-themes/modules-7 for
further information.

But finally, I'm not 100% certain that the module is necessary - could you not just use the default Sales Tax type that the Commerce Tax module provides, or create a new tax type that does not have the "Display taxes of this type inclusive in product prices" option checked?

If you are still interested on being able to opt into security coverage for projects you create, please open a new application using a project for which the only commits (for the time required to set the application's status to Fixed) are from you.
Please don't open a new application if you aren't sure to have time to dedicate to the application, or it will be closed again as won't fix.

I am closing this application due to lack of activity.