From c5ca9ab5a39649d3f838554b403f041d4df5a05b Mon Sep 17 00:00:00 2001
From: Dor Shaked <dorsh@monday.com>
Date: Wed, 7 Oct 2020 11:30:28 +0300
Subject: [PATCH 1/2] feat: bump version * bump version * update README.md for
 release flow

---
 .../FormattedNumber/FormattedNumber-test.js   | 34 +++++++--------
 .../FormattedNumber/FormattedNumber.jsx       |  3 +-
 .../__stories__/formattedNumber.stories.js    | 42 +++++++++----------
 .../LinearProgressBar/Bar/Bar.jsx             |  5 +--
 .../LinearProgressBar-test.js                 |  6 +--
 .../__stories__/linearProgressBar.stories.js  |  2 +-
 6 files changed, 45 insertions(+), 47 deletions(-)

diff --git a/src/components/FormattedNumber/FormattedNumber-test.js b/src/components/FormattedNumber/FormattedNumber-test.js
index 95805476e0..c8c7e8da77 100644
--- a/src/components/FormattedNumber/FormattedNumber-test.js
+++ b/src/components/FormattedNumber/FormattedNumber-test.js
@@ -1,7 +1,7 @@
 import React from "react";
+import { render, cleanup } from "@testing-library/react";
 import { expect } from "../../test/test-helpers";
 import FormattedNumber from "./FormattedNumber";
-import { render, cleanup } from "@testing-library/react";
 
 describe("FormattedNumber Tests", () => {
   beforeEach(() => cleanup());
@@ -9,26 +9,24 @@ describe("FormattedNumber Tests", () => {
 
   describe("emptyPlaceHolder", () => {
     it("no props where provided", () => {
-      const { getByText } = render(<FormattedNumber id={"test"} />);
+      const { getByText } = render(<FormattedNumber id="test" />);
       expect(getByText("N/A")).to.be.ok;
     });
 
     it("invalid value - empty string", () => {
-      const { getByText } = render(<FormattedNumber id={"test"} value={" "} />);
+      const { getByText } = render(<FormattedNumber id="test" value={" "} />);
       expect(getByText("N/A")).to.be.ok;
     });
 
     it("invalid value - mixed content", () => {
-      const { getByText } = render(
-        <FormattedNumber id={"test"} value={"a1s2d"} />
-      );
+      const { getByText } = render(<FormattedNumber id="test" value="a1s2d" />);
       expect(getByText("N/A")).to.be.ok;
     });
 
     it("invalid value - mixed content", () => {
       const emptyPlaceHolderText = "Test";
       const { getByText } = render(
-        <FormattedNumber id={"test"} emptyPlaceHolder={emptyPlaceHolderText} />
+        <FormattedNumber id="test" emptyPlaceHolder={emptyPlaceHolderText} />
       );
       expect(getByText(emptyPlaceHolderText)).to.be.ok;
     });
@@ -42,7 +40,7 @@ describe("FormattedNumber Tests", () => {
     it("should format small number without sign", () => {
       const expectedText = "98";
       const { getByText } = render(
-        <FormattedNumber id={"test"} value={smallNumber} />
+        <FormattedNumber id="test" value={smallNumber} />
       );
       expect(getByText(expectedText)).to.be.ok;
     });
@@ -50,7 +48,7 @@ describe("FormattedNumber Tests", () => {
     it("should format large number with sign", () => {
       const expectedText = "987.6543M";
       const { getByText } = render(
-        <FormattedNumber id={"test"} value={largeNumber} decimalPrecision={4} />
+        <FormattedNumber id="test" value={largeNumber} decimalPrecision={4} />
       );
       expect(getByText(expectedText)).to.be.ok;
     });
@@ -58,7 +56,7 @@ describe("FormattedNumber Tests", () => {
     it("should format large number without sign", () => {
       const expectedText = "987,654,321";
       const { getByText } = render(
-        <FormattedNumber id={"test"} value={largeNumber} compact={false} />
+        <FormattedNumber id="test" value={largeNumber} compact={false} />
       );
       expect(getByText(expectedText)).to.be.ok;
     });
@@ -67,7 +65,7 @@ describe("FormattedNumber Tests", () => {
       const expectedText = "987,654,321.123";
       const { getByText } = render(
         <FormattedNumber
-          id={"test"}
+          id="test"
           value={decimalNumber}
           compact={false}
           decimalPrecision={3}
@@ -80,7 +78,7 @@ describe("FormattedNumber Tests", () => {
       const expectedText = "987,654,321";
       const { getByText } = render(
         <FormattedNumber
-          id={"test"}
+          id="test"
           value={decimalNumber}
           compact={false}
           decimalPrecision={-10}
@@ -93,7 +91,7 @@ describe("FormattedNumber Tests", () => {
       const expectedText = "987,654,321.123456";
       const { getByText } = render(
         <FormattedNumber
-          id={"test"}
+          id="test"
           value={decimalNumber}
           compact={false}
           decimalPrecision={50}
@@ -109,7 +107,7 @@ describe("FormattedNumber Tests", () => {
     it("should use handle unsupported local", () => {
       const badLocal = "bad";
       const { container, getByText } = render(
-        <FormattedNumber id={"test"} value={value} local={badLocal} />
+        <FormattedNumber id="test" value={value} local={badLocal} />
       );
       expect(container).is.ok;
       expect(getByText("456")).to.be.ok;
@@ -122,9 +120,9 @@ describe("FormattedNumber Tests", () => {
     const suffix = "suf";
 
     it("should render predix and suffix in the correct order if provided", () => {
-      const { getByText, container } = render(
+      const { container } = render(
         <FormattedNumber
-          id={"test"}
+          id="test"
           value={value}
           prefix={prefix}
           suffix={suffix}
@@ -138,9 +136,9 @@ describe("FormattedNumber Tests", () => {
     });
 
     it("should render rtl", () => {
-      const { getByText, container } = render(
+      const { container } = render(
         <FormattedNumber
-          id={"test"}
+          id="test"
           value={value}
           prefix={prefix}
           suffix={suffix}
diff --git a/src/components/FormattedNumber/FormattedNumber.jsx b/src/components/FormattedNumber/FormattedNumber.jsx
index 581039139e..de41b1142f 100644
--- a/src/components/FormattedNumber/FormattedNumber.jsx
+++ b/src/components/FormattedNumber/FormattedNumber.jsx
@@ -112,7 +112,8 @@ FormattedNumber.defaultProps = {
   decimalPrecision: 2,
   compact: true,
   local: FormattedNumber.localFallBack,
-  rtl: false
+  rtl: false,
+  className: ""
 };
 
 export default FormattedNumber;
diff --git a/src/components/FormattedNumber/__stories__/formattedNumber.stories.js b/src/components/FormattedNumber/__stories__/formattedNumber.stories.js
index 39fabf93e1..75e49d20c8 100644
--- a/src/components/FormattedNumber/__stories__/formattedNumber.stories.js
+++ b/src/components/FormattedNumber/__stories__/formattedNumber.stories.js
@@ -1,31 +1,31 @@
 import React from "react";
-import FormattedNumber from "../FormattedNumber";
 import { text, number, select, boolean } from "@storybook/addon-knobs";
+import FormattedNumber from "../FormattedNumber";
 import "./formattedNumber.stories.scss";
 
 export const Sandbox = () => (
-    <div style={{ width: "100%", margin: "40px" }}>
-        <div style={{ width: "40px", height: "40px" }}>
-            <FormattedNumber
-                className={select(
-                    "With custom class",
-                    ["formatted-number--custom-class", ""],
-                    ""
-                )}
-                value={number("Value", 50000)}
-                prefix={text("Prefix", "$")}
-                suffix={text("Suffix", "Human")}
-                decimalPrecision={number("Precision", 2)}
-                emptyPlaceHolder={text("Place Holder", "N/A")}
-                local={text("Local", "en-US")}
-                compact={boolean("Is compact", true)}
-                rtl={boolean("Right to Left", false)}
-            />
-        </div>
+  <div style={{ width: "100%", margin: "40px" }}>
+    <div style={{ width: "40px", height: "40px" }}>
+      <FormattedNumber
+        className={select(
+          "With custom class",
+          ["formatted-number--custom-class", ""],
+          ""
+        )}
+        value={number("Value", 50000)}
+        prefix={text("Prefix", "$")}
+        suffix={text("Suffix", "Human")}
+        decimalPrecision={number("Precision", 2)}
+        emptyPlaceHolder={text("Place Holder", "N/A")}
+        local={text("Local", "en-US")}
+        compact={boolean("Is compact", true)}
+        rtl={boolean("Right to Left", false)}
+      />
     </div>
+  </div>
 );
 
 export default {
-    title: "Components/FormattedNumber",
-    component: FormattedNumber
+  title: "Components/FormattedNumber",
+  component: FormattedNumber
 };
diff --git a/src/components/ProgressBars/LinearProgressBar/Bar/Bar.jsx b/src/components/ProgressBars/LinearProgressBar/Bar/Bar.jsx
index d8cab21fd0..c7aa998ffe 100644
--- a/src/components/ProgressBars/LinearProgressBar/Bar/Bar.jsx
+++ b/src/components/ProgressBars/LinearProgressBar/Bar/Bar.jsx
@@ -4,11 +4,10 @@ import { calculatePercentage } from "../LinearProgressBarHelpers";
 
 const Bar = ({ value, baseClass, barStyle, animated, min, max, color }) => {
   const classNames = useMemo(() => {
-    const classNames = cx(baseClass, `${baseClass}--${barStyle}`, {
+    return cx(baseClass, `${baseClass}--${barStyle}`, {
       [`${baseClass}--animate`]: animated
     });
-    return classNames;
-  }, [barStyle, animated, animated]);
+  }, [barStyle, animated, baseClass]);
 
   const valuePercentage = useMemo(() => {
     if (value === null || value === undefined) return 0;
diff --git a/src/components/ProgressBars/LinearProgressBar/LinearProgressBar-test.js b/src/components/ProgressBars/LinearProgressBar/LinearProgressBar-test.js
index eb0773ea5d..2456205fbc 100644
--- a/src/components/ProgressBars/LinearProgressBar/LinearProgressBar-test.js
+++ b/src/components/ProgressBars/LinearProgressBar/LinearProgressBar-test.js
@@ -1,7 +1,7 @@
 import React from "react";
-import { expect, sinon } from "../../../test/test-helpers";
-import LinearProgressBar from "./LinearProgressBar";
 import { render, cleanup, act, screen } from "@testing-library/react";
+import { expect } from "../../../test/test-helpers";
+import LinearProgressBar from "./LinearProgressBar";
 
 describe("ProgressBars Tests", () => {
   let component;
@@ -76,7 +76,7 @@ describe("ProgressBars Tests", () => {
       const value = 13;
       for (let i = 0; i < 2; i++) {
         act(() => {
-          component = rerender(
+          rerender(
             <LinearProgressBar
               value={value + i}
               max={100}
diff --git a/src/components/ProgressBars/LinearProgressBar/__stories__/linearProgressBar.stories.js b/src/components/ProgressBars/LinearProgressBar/__stories__/linearProgressBar.stories.js
index dea3d47988..f26091c1e8 100644
--- a/src/components/ProgressBars/LinearProgressBar/__stories__/linearProgressBar.stories.js
+++ b/src/components/ProgressBars/LinearProgressBar/__stories__/linearProgressBar.stories.js
@@ -1,7 +1,7 @@
 import React from "react";
+import { boolean, number, select, color } from "@storybook/addon-knobs";
 import LinearProgressBar from "../LinearProgressBar";
 import colors from "../../../../constants/colors.json";
-import { boolean, number, select, color } from "@storybook/addon-knobs";
 import "./linearProgressBar.stories.scss";
 
 export const Sandbox = () => (

From ed24b9713ad63d910eca32616bb6b16af42cad18 Mon Sep 17 00:00:00 2001
From: Dor Shaked <dorsh@monday.com>
Date: Wed, 7 Oct 2020 11:33:42 +0300
Subject: [PATCH 2/2] feat: bump version * bump version

---
 package-lock.json | 2 +-
 package.json      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index c605290574..8a5e9b654c 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,6 +1,6 @@
 {
   "name": "monday-ui-react-core",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "lockfileVersion": 1,
   "requires": true,
   "dependencies": {
diff --git a/package.json b/package.json
index 246f478f88..f93fc94f26 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "monday-ui-react-core",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "description": "Official monday.com UI resources for application development in React.js",
   "main": "dist/main.js",
   "scripts": {