adobe / commerce-cif-connector

AEM Commerce connector for Magento and GraphQL
Apache License 2.0
43 stars 27 forks source link

CIF-1027 - Clean up occurrences of ccif in commerce-cif-connector project #50

Closed LSantha closed 5 years ago

LSantha commented 5 years ago

There are several occurrences of the old acronym CCIF in commerce-cif-connector as "ccif" in component names and variable names which is now obsolete and misleading and it should be replaced with cif.

Description

I've replaced occurrences of ccif with cif.

Related Issue

CIF-1027

Motivation and Context

Consistent naming, fixing deprecated naming patterns.

How Has This Been Tested?

Manually.

Types of changes

Checklist:

codecov[bot] commented 5 years ago

Codecov Report

Merging #50 into master will increase coverage by 0.17%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #50      +/-   ##
============================================
+ Coverage      85.5%   85.67%   +0.17%     
  Complexity      314      314              
============================================
  Files            18       18              
  Lines          1145     1145              
  Branches        178      178              
============================================
+ Hits            979      981       +2     
+ Misses           77       75       -2     
  Partials         89       89
Flag Coverage Δ Complexity Δ
#integration 61.92% <ø> (+0.17%) 215 <ø> (ø) :arrow_down:
#unittests 79.63% <ø> (ø) 292 <ø> (ø) :arrow_down:
Impacted Files Coverage Δ Complexity Δ
...ommerce/graphql/resource/RootCategoryResource.java 95% <ø> (ø) 9 <0> (ø) :arrow_down:
...e/cq/commerce/graphql/resource/ResourceMapper.java 79.19% <0%> (+1.15%) 35% <0%> (ø) :arrow_down:

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 02a7080...3c60627. Read the comment docs.