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

refactor(api-request-builder): rename createService argument #1524

Merged
merged 1 commit into from
Apr 6, 2020

Conversation

jherey
Copy link
Contributor

@jherey jherey commented Apr 6, 2020

Summary

Renamed options to projectKey

Description

Changed the 2nd argument to createService function to projectKey. This makes it easy to read the code and understand, as against options which means an object.
affects: @commercetools/api-request-builder

@codecov
Copy link

codecov bot commented Apr 6, 2020

Codecov Report

Merging #1524 into master will not change coverage by %.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1524   +/-   ##
=======================================
  Coverage   98.62%   98.62%           
=======================================
  Files         128      128           
  Lines        3281     3281           
  Branches      757      757           
=======================================
  Hits         3236     3236           
  Misses         41       41           
  Partials        4        4           
Impacted Files Coverage Δ
packages/api-request-builder/src/create-service.js 100.00% <100.00%> (ø)

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 bba6e26...f96eed7. Read the comment docs.

Copy link
Contributor

@daern91 daern91 left a comment

Choose a reason for hiding this comment

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

Another nice little find, thanks @jherey!

@jherey jherey force-pushed the 1523-rename-create-service-argument branch from 76eb9ef to e5c03e0 Compare April 6, 2020 11:58
@daern91 daern91 changed the title refactor(create-service): rename createService argument refactor(api-request-builder): rename createService argument Apr 6, 2020
Copy link
Contributor

@danrleyt danrleyt left a comment

Choose a reason for hiding this comment

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

Good catch!

Copy link
Contributor

@katmatt katmatt left a comment

Choose a reason for hiding this comment

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

You have eyes sharp like an 🦅 Looks very good to me 👍

affects: @commercetools/api-request-builder

the second argument for createService is changed to projectKey

ISSUES CLOSED: #1523
@jherey jherey force-pushed the 1523-rename-create-service-argument branch from e5c03e0 to f96eed7 Compare April 6, 2020 12:54
@jherey jherey merged commit ec6edd6 into master Apr 6, 2020
@jherey jherey deleted the 1523-rename-create-service-argument branch April 6, 2020 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants