refactor(make-breakpoints): remove safe-make-breakpoints

rename safe-make-breakpoints with make-breakpoints

BREAKING CHANGE: make breakpoints function no longer
returns the Maybe type. The associative array is returned

__tests__/safe-make-breakpoints/base-prop-process.spec.js → __tests__/make-breakpoints/base-prop-process.spec.js

@@ -1,4 +1,4 @@
-import { basePropProcess } from '../../src/safe-make-breakpoints/base-prop-process';
+import { basePropProcess } from '../../src/make-breakpoints/base-prop-process';
 
 describe('basePropProcess', () => {
   it('breakpoint object with a base key containing an array of numbers', () => {

__tests__/safe-make-breakpoints/create-breakpoints.spec.js → __tests__/make-breakpoints/create-breakpoints.spec.js

@@ -1,4 +1,4 @@
-import { createBreakpoints } from '../../src/safe-make-breakpoints/create-breakpoints';
+import { createBreakpoints } from '../../src/make-breakpoints/create-breakpoints';
 import { userConfig } from '../../src/models/user-config';
 
 describe('createBreakpoints', () => {

__tests__/safe-make-breakpoints/index.spec.js → __tests__/make-breakpoints/index.spec.js

@@ -1,10 +1,10 @@
 import { userConfig } from '../../src/models/user-config';
-import { safeMakeBreakpoints } from '../../src/safe-make-breakpoints';
+import { makeBreakpoints } from '../../src/make-breakpoints';
 
 describe('make breakpoints', () => {
-  describe('safeMakeBreakpoints', () => {
+  describe('makeBreakpoints', () => {
     it('create a list of breakpoint values if the user config is valid', () => {
-      expect(safeMakeBreakpoints(userConfig).getOr()).toEqual({
+      expect(makeBreakpoints(userConfig)).toEqual({
         default: {
           base: [16, 32],
           lineHeight: 1.5,

__tests__/safe-make-breakpoints/inherit-props.spec.js → __tests__/make-breakpoints/inherit-props.spec.js

@@ -1,4 +1,4 @@
-import { inheritProps } from '../../src/safe-make-breakpoints/inherit-props';
+import { inheritProps } from '../../src/make-breakpoints/inherit-props';
 
 describe('inheritProps', () => {
   it('inherits all missing key values', () => {

__tests__/safe-make-breakpoints/make-break-map.spec.js → __tests__/make-breakpoints/make-break-map.spec.js

@@ -1,4 +1,4 @@
-import { makeBreaksMap } from '../../src/safe-make-breakpoints/make-breaks-map';
+import { makeBreaksMap } from '../../src/make-breakpoints/make-breaks-map';
 import { breakpoints } from '../../src/models/breakpoints';
 
 describe('makeBreaksMap', () => {

__tests__/safe-make-breakpoints/make-breakpoints-process.spec.js → __tests__/make-breakpoints/make-breakpoints-process.spec.js

@@ -1,4 +1,4 @@
-import { makeBreakpointsProcess } from '../../src/safe-make-breakpoints/make-breakpoints-process';
+import { makeBreakpointsProcess } from '../../src/make-breakpoints/make-breakpoints-process';
 import { userConfig } from '../../src/models/user-config';
 import { breakpointsMap } from '../../src/models/breakpoints';
 

__tests__/safe-make-breakpoints/make-default-break.spec.js → __tests__/make-breakpoints/make-default-break.spec.js

@@ -1,4 +1,4 @@
-import { makeDefaultBreak } from '../../src/safe-make-breakpoints/make-default-break';
+import { makeDefaultBreak } from '../../src/make-breakpoints/make-default-break';
 import { userConfig } from '../../src/models/user-config';
 
 describe('makeDefaultBreak', () => {

__tests__/safe-make-breakpoints/make-named-breaks.spec.js → __tests__/make-breakpoints/make-named-breaks.spec.js

@@ -1,4 +1,4 @@
-import { makeNamedBreaks } from '../../src/safe-make-breakpoints/make-named-breaks';
+import { makeNamedBreaks } from '../../src/make-breakpoints/make-named-breaks';
 import { userConfig } from '../../src/models/user-config';
 
 describe('makeNamedBreaks', () => {

__tests__/safe-make-breakpoints/ratio-prop-utils.spec.js → __tests__/make-breakpoints/ratio-prop-utils.spec.js

@@ -3,7 +3,7 @@ import {
   makeFontSizeFromLiteral,
   calcRatio,
   calcRatioProcess,
-} from '../../src/safe-make-breakpoints/ratio-prop-utils';
+} from '../../src/make-breakpoints/ratio-prop-utils';
 
 const ratioLiteral = '45px at 6';
 

__tests__/safe-make-breakpoints/reaname-prop.spec.js → __tests__/make-breakpoints/reaname-prop.spec.js

@@ -1,4 +1,4 @@
-import { renameProp } from '../../src/safe-make-breakpoints/rename-prop';
+import { renameProp } from '../../src/make-breakpoints/rename-prop';
 
 describe('renameProp', () => {
   it('return object with ranamed prop', () => {

__tests__/safe-make-breakpoints/root-prop-utils.spec.js → __tests__/make-breakpoints/root-prop-utils.spec.js

@@ -1,7 +1,7 @@
 import {
   calcRoot,
   setPropRoot,
-} from '../../src/safe-make-breakpoints/root-prop-utils';
+} from '../../src/make-breakpoints/root-prop-utils';
 
 describe('calcRoot', () => {
   it('calculated root', () => {

__tests__/safe-make-breakpoints/to-px-break-value.spec.js → __tests__/make-breakpoints/to-px-break-value.spec.js

@@ -1,4 +1,4 @@
-import { toPxBreakValue } from '../../src/safe-make-breakpoints/to-px-break-value';
+import { toPxBreakValue } from '../../src/make-breakpoints/to-px-break-value';
 
 describe('toPxBreakValue', () => {
   it('returns the breakpoints value converted from em to px', () => {

src/index.js

@@ -1,7 +1,7 @@
 // @flow
 
 export { modularScale } from './calculators/modular-scale';
-export { safeMakeBreakpoints } from './safe-make-breakpoints';
+export { makeBreakpoints } from './make-breakpoints';
 export * as constants from './constants';
 export { camelize } from './convertors/camelize';
 export { normalizeString } from './convertors/normalize-string';

src/safe-make-breakpoints/index.js → src/make-breakpoints/index.js

@@ -1,6 +1,5 @@
 // @flow
-import * as R from 'ramda';
-import { Just, Nothing } from 'igogo';
+
 import memoizeone from 'memoize-one';
 import { makeBreakpointsProcess } from './make-breakpoints-process';
 import { validateUserConfig } from '../validate-user-config';
@@ -11,11 +10,11 @@ const memoizedMakeBreakpoints: (UserConfig) => BreakpointsMap = memoizeone(
   makeBreakpointsProcess,
 );
 
-export const safeMakeBreakpoints: (UserConfig) => * = R.ifElse(
-  validateUserConfig,
-  R.compose(
-    Just,
-    memoizedMakeBreakpoints,
-  ),
-  Nothing,
-);
+/* eslint-disable consistent-return */
+export const makeBreakpoints: (UserConfig) => * = (config) => {
+  if (validateUserConfig(config)) {
+    return memoizedMakeBreakpoints(config);
+  }
+};
+
+/* eslint-enable */