Skip to content
This repository has been archived by the owner on Jan 6, 2025. It is now read-only.

Add string functions #87

Merged
merged 2 commits into from
Sep 3, 2017
Merged

Add string functions #87

merged 2 commits into from
Sep 3, 2017

Conversation

frinyvonnick
Copy link
Contributor

Issue: #79

  • capitalize
  • replace
  • toLower
  • toUpper

@frinyvonnick frinyvonnick requested a review from nlepage September 3, 2017 09:58
@frinyvonnick frinyvonnick changed the title ✨ Add string functions Add string functions Sep 3, 2017
@codecov-io
Copy link

codecov-io commented Sep 3, 2017

Codecov Report

Merging #87 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #87   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          59     63    +4     
  Lines         120    124    +4     
  Branches        7      7           
=====================================
+ Hits          120    124    +4
Impacted Files Coverage Δ
src/string/replace.js 100% <100%> (ø)
src/string/capitalize.js 100% <100%> (ø)
src/string/toUpper.js 100% <100%> (ø)
src/string/toLower.js 100% <100%> (ø)

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 bfa9cba...6b104a3. Read the comment docs.

@nlepage nlepage added this to the 0.3-alpha milestone Sep 3, 2017
Copy link
Member

@nlepage nlepage left a comment

Choose a reason for hiding this comment

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

Add string namespace to ChainWrapper and UsingWrapper

* @return {Object} Returns the updated object.
* @example capitalize({ nested: { a: "a string" } }, 'nested.a') // => { nested: { a: "A string" } }
* @see {@link https://lodash.com/docs#capitalize|lodash.capitalize} for more information.
* @see {@link https://lodash.com/docs#identity|lodash.identity} for more information.
Copy link
Member

Choose a reason for hiding this comment

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

Remove this link


describe('Capitalize', () => {

it('should converts the first character of string to upper case and the remaining to lower case', () => {
Copy link
Member

Choose a reason for hiding this comment

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

"should convert ..." -s

* @return {Object} Returns the updated object.
* @example replace({ nested: { a: "Hi Nico" } }, 'nested.a', 'Nico', 'Yvo') // => { nested: { a: "Hi Yvo" } }
* @see {@link https://lodash.com/docs#replace|lodash.replace} for more information.
* @see {@link https://lodash.com/docs#identity|lodash.identity} for more information.
Copy link
Member

Choose a reason for hiding this comment

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

Remove this link

* @return {Object} Returns the updated object.
* @example toLower({ nested: { a: "A STRING" } }, 'nested.a') // => { nested: { a: "a string" } }
* @see {@link https://lodash.com/docs#toLower|lodash.toLower} for more information.
* @see {@link https://lodash.com/docs#identity|lodash.identity} for more information.
Copy link
Member

Choose a reason for hiding this comment

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

Remove this link

import { convert } from 'util/convert'

/**
* Converts string, as a whole, to lower case just like String#toLowerCase.
Copy link
Member

Choose a reason for hiding this comment

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


describe('toLower', () => {

it('should converts string, as a whole, to lower case just like String#toLowerCase', () => {
Copy link
Member

Choose a reason for hiding this comment

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

"should convert" -s

* @return {Object} Returns the updated object.
* @example toUpper({ nested: { a: "a string" } }, 'nested.a') // => { nested: { a: "A STRING" } }
* @see {@link https://lodash.com/docs#toUpper|lodash.toUpper} for more information.
* @see {@link https://lodash.com/docs#identity|lodash.identity} for more information.
Copy link
Member

Choose a reason for hiding this comment

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

Remove this link


describe('toUpper', () => {

it('should converts string, as a whole, to upper case just like String#toUpperCase', () => {
Copy link
Member

Choose a reason for hiding this comment

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

"should convert" -s

import { convert } from 'util/convert'

/**
* Converts string, as a whole, to upper case just like String#toUpperCase.
Copy link
Member

Choose a reason for hiding this comment

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

@nlepage nlepage merged commit 45ab247 into master Sep 3, 2017
@nlepage nlepage deleted the feature/79_string branch September 3, 2017 15:11
nlepage pushed a commit that referenced this pull request Sep 3, 2017
nlepage pushed a commit that referenced this pull request Sep 3, 2017
* @since 0.3.0
*/
export {
capizalize,
Copy link
Contributor

Choose a reason for hiding this comment

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

👎

Choose a reason for hiding this comment

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

🍺

Choose a reason for hiding this comment

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

💩

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants