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

Simple sugar text correction in Mock Function docs #8406

Merged
merged 3 commits into from May 2, 2019

Conversation

Projects
None yet
4 participants
@morganmccol
Copy link
Contributor

commented May 1, 2019

Summary

A few places in the Mock Function docs refer to "simple sugar" functions. I've updated these to syntactic sugar to match the correct terminology.

@codecov-io

This comment has been minimized.

Copy link

commented May 1, 2019

Codecov Report

Merging #8406 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #8406   +/-   ##
=======================================
  Coverage   62.37%   62.37%           
=======================================
  Files         266      266           
  Lines       10739    10739           
  Branches     2614     2615    +1     
=======================================
  Hits         6698     6698           
  Misses       3459     3459           
  Partials      582      582

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 a9c0e6b...5f3bb73. Read the comment docs.

@SimenB

SimenB approved these changes May 2, 2019

Copy link
Collaborator

left a comment

Good call! Left a suggestion, happy to merge after that 🙂

Show resolved Hide resolved docs/MockFunctionAPI.md Outdated

@SimenB SimenB merged commit 1c5d830 into facebook:master May 2, 2019

11 checks passed

ci/circleci: lint-and-typecheck Your tests passed on CircleCI!
Details
ci/circleci: test-browser Your tests passed on CircleCI!
Details
ci/circleci: test-jest-circus Your tests passed on CircleCI!
Details
ci/circleci: test-node-10 Your tests passed on CircleCI!
Details
ci/circleci: test-node-11 Your tests passed on CircleCI!
Details
ci/circleci: test-node-6 Your tests passed on CircleCI!
Details
ci/circleci: test-node-8 Your tests passed on CircleCI!
Details
ci/circleci: test-or-deploy-website Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
deploy/netlify Deploy preview ready!
Details
facebook.jest #20190502.14 succeeded
Details
@SimenB

This comment has been minimized.

Copy link
Collaborator

commented May 2, 2019

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.