Naming changes

Author: Timer

Diff for: packages/react-scripts/fixtures/kitchensink/src/App.js

@@ -8,23 +8,23 @@ class BuiltEmitter extends React.Component {
   componentDidMount() {
     const { feature } = this.props
     if (!Component.isPrototypeOf(feature)) {
-      this.notifyRendered();
+      this.handleReady();
     }
   }
 
-  notifyRendered() {
+  handleReady() {
     document.dispatchEvent(new Event('ReactFeatureDidMount'));
   }
 
   render() {
     const {
       props: { feature },
-      notifyRendered
+      handleReady
     } = this;
     return (
       <div>
         {createElement(feature, {
-          notifyRendered
+          onReady: handleReady
         })}
       </div>
     );

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/env/NodePath.js

@@ -3,7 +3,7 @@ import load from 'absoluteLoad'
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -17,7 +17,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/env/NodePath.test.js

@@ -6,7 +6,7 @@ describe('NODE_PATH', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<NodePath notifyRendered={resolve} />, div);
+      ReactDOM.render(<NodePath onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ArrayDestructuring.js

@@ -11,7 +11,7 @@ function load() {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -25,7 +25,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ArrayDestructuring.test.js

@@ -6,7 +6,7 @@ describe('array destructuring', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<ArrayDestructuring notifyRendered={resolve} />, div);
+      ReactDOM.render(<ArrayDestructuring onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ArraySpread.js

@@ -11,7 +11,7 @@ function load(users) {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -25,7 +25,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ArraySpread.test.js

@@ -6,7 +6,7 @@ describe('array spread', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<ArraySpread notifyRendered={resolve} />, div);
+      ReactDOM.render(<ArraySpread onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/AsyncAwait.js

@@ -11,7 +11,7 @@ async function load() {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -25,7 +25,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/AsyncAwait.test.js

@@ -6,7 +6,7 @@ describe('async/await', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<AsyncAwait notifyRendered={resolve} />, div);
+      ReactDOM.render(<AsyncAwait onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ClassProperties.js

@@ -2,7 +2,7 @@ import React, { Component, PropTypes } from 'react'
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   users = [
@@ -13,7 +13,7 @@ export default class extends Component {
   ];
 
   componentDidMount() {
-    this.props.notifyRendered()
+    this.props.onReady()
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ClassProperties.test.js

@@ -6,7 +6,7 @@ describe('class properties', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<ClassProperties notifyRendered={resolve} />, div);
+      ReactDOM.render(<ClassProperties onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ComputedProperties.js

@@ -11,7 +11,7 @@ function load(prefix) {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -25,7 +25,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ComputedProperties.test.js

@@ -6,7 +6,7 @@ describe('computed properties', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<ComputedProperties notifyRendered={resolve} />, div);
+      ReactDOM.render(<ComputedProperties onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/CustomInterpolation.js

@@ -16,7 +16,7 @@ function load() {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -30,7 +30,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/CustomInterpolation.test.js

@@ -6,7 +6,7 @@ describe('custom interpolation', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<CustomInterpolation notifyRendered={resolve} />, div);
+      ReactDOM.render(<CustomInterpolation onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/DefaultParameters.js

@@ -11,7 +11,7 @@ function load(id = 0) {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -25,7 +25,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/DefaultParameters.test.js

@@ -6,7 +6,7 @@ describe('default parameters', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<DefaultParameters notifyRendered={resolve} />, div);
+      ReactDOM.render(<DefaultParameters onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/DestructuringAndAwait.js

@@ -11,7 +11,7 @@ async function load() {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -25,7 +25,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/DestructuringAndAwait.test.js

@@ -6,7 +6,7 @@ describe('destructuring and await', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<DestructuringAndAwait notifyRendered={resolve} />, div);
+      ReactDOM.render(<DestructuringAndAwait onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/Generators.js

@@ -10,7 +10,7 @@ function * load(limit) {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -27,7 +27,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/Generators.test.js

@@ -6,7 +6,7 @@ describe('generators', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<Generators notifyRendered={resolve} />, div);
+      ReactDOM.render(<Generators onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ObjectDestructuring.js

@@ -11,7 +11,7 @@ function load() {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -25,7 +25,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ObjectDestructuring.test.js

@@ -6,7 +6,7 @@ describe('object destructuring', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<ObjectDestructuring notifyRendered={resolve} />, div);
+      ReactDOM.render(<ObjectDestructuring onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ObjectSpread.js

@@ -11,7 +11,7 @@ function load(baseUser) {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -25,7 +25,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/ObjectSpread.test.js

@@ -6,7 +6,7 @@ describe('object spread', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<ObjectSpread notifyRendered={resolve} />, div);
+      ReactDOM.render(<ObjectSpread onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/Promises.js

@@ -11,7 +11,7 @@ function load() {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -26,7 +26,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/Promises.test.js

@@ -6,7 +6,7 @@ describe('promises', () => {
   it('renders without crashing', () => {
     const div = document.createElement('div');
     return new Promise(resolve => {
-      ReactDOM.render(<Promises notifyRendered={resolve} />, div);
+      ReactDOM.render(<Promises onReady={resolve} />, div);
     });
   });
 });

Diff for: packages/react-scripts/fixtures/kitchensink/src/features/syntax/RestAndDefault.js

@@ -11,7 +11,7 @@ function load({ id, ...rest } = { id: 0, user: { id: 42, name: '42' } }) {
 
 export default class extends Component {
   static propTypes = {
-    notifyRendered: PropTypes.func.isRequired
+    onReady: PropTypes.func.isRequired
   }
 
   constructor(props) {
@@ -25,7 +25,7 @@ export default class extends Component {
   }
 
   componentDidUpdate() {
-    this.props.notifyRendered();
+    this.props.onReady();
   }
 
   render() {