Skip to content

Commit

Permalink
fix lint errors
Browse files Browse the repository at this point in the history
We had to choose between disabling react/prop-types or react/no-unused-prop-types.

Since react/no-unused-prop-types has a bug (see jsx-eslint/eslint-plugin-react#811), we're disabling it for now.
  • Loading branch information
jeffreywescott authored and jeffreywescott committed Feb 16, 2017
1 parent 603a597 commit 9f93796
Show file tree
Hide file tree
Showing 9 changed files with 10 additions and 0 deletions.
1 change: 1 addition & 0 deletions common/containers/ChapterForm/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ class ChapterFormContainer extends Component {
ChapterFormContainer.propTypes = {
chapter: PropTypes.object,
isBusy: PropTypes.bool.isRequired,
loading: PropTypes.bool.isRequired,
fetchData: PropTypes.func.isRequired,
showLoad: PropTypes.func.isRequired,
hideLoad: PropTypes.func.isRequired,
Expand Down
1 change: 1 addition & 0 deletions common/containers/ChapterList/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ class ChapterListContainer extends Component {
ChapterListContainer.propTypes = {
chapters: PropTypes.array.isRequired,
isBusy: PropTypes.bool.isRequired,
loading: PropTypes.bool.isRequired,
currentUser: PropTypes.object.isRequired,
fetchData: PropTypes.func.isRequired,
navigate: PropTypes.func.isRequired,
Expand Down
1 change: 1 addition & 0 deletions common/containers/ProjectDetail/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ ProjectDetailContainer.propTypes = {
projectEvaluations: PropTypes.array,
projectUserSummaries: PropTypes.array,
isBusy: PropTypes.bool.isRequired,
loading: PropTypes.bool.isRequired,
currentUser: PropTypes.object,
fetchData: PropTypes.func.isRequired,
navigate: PropTypes.func.isRequired,
Expand Down
1 change: 1 addition & 0 deletions common/containers/ProjectForm/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ class ProjectFormContainer extends Component {
ProjectFormContainer.propTypes = {
project: PropTypes.object,
isBusy: PropTypes.bool.isRequired,
loading: PropTypes.bool.isRequired,
fetchData: PropTypes.func.isRequired,
showLoad: PropTypes.func.isRequired,
hideLoad: PropTypes.func.isRequired,
Expand Down
1 change: 1 addition & 0 deletions common/containers/ProjectList/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ class ProjectListContainer extends Component {
ProjectListContainer.propTypes = {
projects: PropTypes.array.isRequired,
isBusy: PropTypes.bool.isRequired,
loading: PropTypes.bool.isRequired,
currentUser: PropTypes.object.isRequired,
fetchData: PropTypes.func.isRequired,
navigate: PropTypes.func.isRequired,
Expand Down
1 change: 1 addition & 0 deletions common/containers/RetroSurvey/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -261,6 +261,7 @@ class RetroSurveyContainer extends Component {

RetroSurveyContainer.propTypes = {
isBusy: PropTypes.bool.isRequired,
loading: PropTypes.bool.isRequired,
currentUser: PropTypes.object,
error: PropTypes.object,
groupIndex: PropTypes.number,
Expand Down
2 changes: 2 additions & 0 deletions common/containers/UserDetail/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,8 @@ class UserDetailContainer extends Component {
UserDetailContainer.propTypes = {
user: PropTypes.object,
userProjectSummaries: PropTypes.array,
isBusy: PropTypes.bool.isRequired,
loading: PropTypes.bool.isRequired,
fetchData: PropTypes.func.isRequired,
navigate: PropTypes.func.isRequired,
showLoad: PropTypes.func.isRequired,
Expand Down
1 change: 1 addition & 0 deletions common/containers/UserList/index.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ class UserListContainer extends Component {
UserListContainer.propTypes = {
users: PropTypes.array.isRequired,
isBusy: PropTypes.bool.isRequired,
loading: PropTypes.bool.isRequired,
currentUser: PropTypes.object.isRequired,
fetchData: PropTypes.func.isRequired,
navigate: PropTypes.func.isRequired,
Expand Down
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -219,6 +219,7 @@
"single",
"avoid-escape"
],
"react/no-unused-prop-types": 0,
"react/forbid-component-props": 0,
"react/jsx-indent": [
2,
Expand Down

0 comments on commit 9f93796

Please sign in to comment.