Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

MM-22155 ESLint configuration updates part 1 #5544

Merged
merged 7 commits into from
May 25, 2020
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
MM-22155 Disable no-magic-numbers
  • Loading branch information
hmhealey committed May 19, 2020
commit f642b93962cd5a60aeb1e4ce917085b8153ed543
13 changes: 0 additions & 13 deletions .eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,19 +38,6 @@
]
}
],
"no-magic-numbers": [
1,
{
"ignore": [
-1,
0,
1,
2
],
"enforceConst": true,
"detectObjects": true
}
],
"no-undefined": 0,
"react/jsx-filename-extension": 0,
"react/prop-types": [
Expand Down
2 changes: 1 addition & 1 deletion components/searchable_user_list/searchable_user_list.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ class SearchableUserList extends React.Component {

static defaultProps = {
users: [],
usersPerPage: 50, // eslint-disable-line no-magic-numbers
usersPerPage: 50,
extraInfo: {},
actions: [],
actionProps: {},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -61,8 +61,8 @@ describe('Verify Accessibility Support in Popovers', () => {
});

// * Verify emoji navigation using arrow keys
verifyArrowKeysEmojiNavigation('{rightarrow}', 3); // eslint-disable-line no-magic-numbers
verifyArrowKeysEmojiNavigation('{leftarrow}', 2); // eslint-disable-line no-magic-numbers
verifyArrowKeysEmojiNavigation('{rightarrow}', 3);
verifyArrowKeysEmojiNavigation('{leftarrow}', 2);

// # Press tab
cy.get($el).focus().tab();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ describe('Keyboard shortcut for adding reactions to last message in channel or t

// # Create a new channel for later use such as when channel is empty test
cy.getCurrentTeamId().then((teamId) => {
// eslint-disable-next-line no-magic-numbers
cy.apiCreateChannel(teamId, newChannelName, newChannelName).then((response) => {
testChannel = response.body;
});
Expand Down
1 change: 0 additions & 1 deletion utils/constants.jsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
// See LICENSE.txt for license information.

/* eslint-disable no-magic-numbers */
import keyMirror from 'key-mirror';

import Permissions from 'mattermost-redux/constants/permissions';
Expand Down