Skip to content

Conversation

kobenguyent
Copy link
Collaborator

Motivation/Description of the PR

  • Fix typos and rephrase some places.

Type of change

  • 🔨 Markdown files fix - not related to source code

pablopaul
pablopaul previously approved these changes Apr 1, 2020
@kobenguyent kobenguyent changed the title fix some typos (fix): some typos in custom-helpers.md Apr 2, 2020
@@ -327,4 +327,4 @@ Retry rules are available in array `recorder.retries`. The last retry rule can b

## Using Typescript

When using typescript, replace `module.exports` with `export` for autocompletion.
When using Typescript, replacing `module.exports` with `export` for autocompletion.
Copy link
Collaborator

Choose a reason for hiding this comment

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

maybe when use Typescript, replace ?

elukoyanov
elukoyanov previously approved these changes Apr 2, 2020
Copy link
Contributor

@DavertMik DavertMik left a comment

Choose a reason for hiding this comment

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

Nice. Maybe you have good ideas how to extend this?
I have a feeling that lots of things are missing here...

@@ -3,13 +3,13 @@ permalink: /helpers
title: Custom Helpers
---

# Extending CodeceptJS with Custom Helopers
# Extending CodeceptJS with custom helpers
Copy link
Contributor

Choose a reason for hiding this comment

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

in title all words must be capitalized

@kobenguyent kobenguyent dismissed stale reviews from elukoyanov and pablopaul via 815b6c7 April 3, 2020 07:51
@github-actions
Copy link

github-actions bot commented Apr 3, 2020

🌀 Tests overview by Testomatio

Found 946 mocha tests in 74 files
No new tests added or removed

⚠️ List all skipped tests (1)
📎 List all suites (62)

@DavertMik DavertMik merged commit 1957d3d into master Apr 5, 2020
@DavertMik DavertMik deleted the PeterNgTr-patch-1 branch April 5, 2020 21:28
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.

4 participants