-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Code Quality Update #2033
Open
Mil4n0r
wants to merge
167
commits into
master
Choose a base branch
from
Mil4n0r/code_quality_monorepo
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Code Quality Update #2033
Changes from all commits
Commits
Show all changes
167 commits
Select commit
Hold shift + click to select a range
5bffdcd
Basic Husky + ESLint + Prettier configuration
Mil4n0r 30b4fcf
Fixed incorrect values in eslint config
Mil4n0r 8a04180
Fixed TS problems within ESLint
Mil4n0r 5ce33b2
Fixed some of the errors detected by eslint
Mil4n0r c9b0e19
Converted configuration files to TS
Mil4n0r d32c8e9
Added security plugin to ESLint
Mil4n0r 2c0a962
Removed unwanted comment
Mil4n0r e424fbc
Fixing ESLint detected problems
Mil4n0r 9bc889f
Fixing ESLint detected problems
Mil4n0r 4552df1
Fixing ESLint detected problems
Mil4n0r db358cf
Fixing ESLint detected problems
Mil4n0r ffbb1d7
Run prettier in the whole src/
Mil4n0r 4a16152
Fixing ESLint detected problems
Mil4n0r 260206f
Moved accessibility files
Mil4n0r f24aa79
Changed tsconfig rootDir
Mil4n0r 5137f2a
Modified TS config to allow jest configs
Mil4n0r f591059
Fixed minor bugs after refactoring
Mil4n0r a470dbe
Relocated accessibility config files
Mil4n0r 966289c
Removed rootDir from tsconfig
Mil4n0r 9c77af7
Changed mocks from js to ts
Mil4n0r 4e03fce
Modified certain incorrect default blocks
Mil4n0r 26763e3
Fixed checkbox color
Mil4n0r e23ffbe
Minor fixes
Mil4n0r 361e606
Removed ESLint rule comment
Mil4n0r 8a6e16c
Removed unexpected end of line
Mil4n0r e5bf87c
Fixed key code typo
Mil4n0r 7c795f1
Removed unnecessary 'break'
Mil4n0r 1ff6f24
Refactored some if/else blocks
Mil4n0r d2bc8f5
Refactored certain exports/imports
Mil4n0r f05db8f
Added pending disabledRule
Mil4n0r eceb4ff
Removed unneeded .eslintrc rules
Mil4n0r 1076feb
Changed .prettierrc config
Mil4n0r 9648aa9
Changed .prettierrc config
Mil4n0r a8bfd61
Removed unwanted change in website
Mil4n0r fbc36f7
Changed .prettierrc config
Mil4n0r f87233c
Changed if/else structure
Mil4n0r c69417a
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 88fe54c
Changed default export name
Mil4n0r 0f5dd74
Fixed default export disabledRules
Mil4n0r 0400789
Fix problem in container style
Mil4n0r dac50f5
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 96a367c
Modified webpack config to match ES6 and TS
Mil4n0r e6e7ebd
Changed babel configuration format to match prettier
Mil4n0r 50ce94e
Reverted Chip changes to types
Mil4n0r acf6f7d
Moved webpack to devDependencies
Mil4n0r 03280ff
Merge branch 'master' into Mil4n0r/code_quality
Mil4n0r 8f50082
Fixed radix bug
Mil4n0r 1525c77
Rollback useEffect dependencies
Mil4n0r 71a999e
Removed unneeded console.log
Mil4n0r 000ee47
Removed font import in storybook preview
Mil4n0r e912d89
Added generated IDs for keys over indexes
Mil4n0r 4e654e8
Fixed problem with v4 typing
Mil4n0r 6678b93
Solved conflicts
Mil4n0r 99fa7cf
Restored changes to key ids that were causing errors in the tests
Mil4n0r 823d8e6
Updated package-lock
Mil4n0r 7bfced3
Merge branch 'master' into Mil4n0r/code_quality
Mil4n0r 4a81138
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 3cf4e29
Merge branch 'Mil4n0r/code_quality' of github.com:dxc-technology/hals…
Mil4n0r b1fca48
Merge branch 'master' into Mil4n0r/code_quality
Mil4n0r 7760e0e
Fixed error in conflicts
Mil4n0r a9f112e
Restored keys
Mil4n0r 88d2755
Removed unneeded dev dependency
Mil4n0r 8508d78
Fixed problem with dependencies
Mil4n0r d511437
Removed build storybook log
Mil4n0r ce30cea
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 4d6048d
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r bfe2073
Chromatic flow improvement
Mil4n0r ecf1799
Updated storybook
Mil4n0r 1c33de8
Merged changes from monorepo update
Mil4n0r 42f352e
Removed repeated import
Mil4n0r 5ee7850
Fixed import in Icon accessibility test
Mil4n0r 7eb0369
Fixed chromatic settings and Breadcrumbs import
Mil4n0r b6f685e
Fixed problem with DXC Icon
Mil4n0r 3051bcc
Refactoring configuration files
Mil4n0r f5ab5a5
Added Eslint recommendations
Mil4n0r 6afb630
Updated package.json
Mil4n0r 5fb4784
Merge changes and solve remaining conflicts
Mil4n0r 3df3af1
Update package-lock
Mil4n0r ba5296e
Fixed problem resolving css from BarChart
Mil4n0r bc42a72
Added husky
Mil4n0r b9d64d2
Added lint-staged
Mil4n0r 03b87a8
Fixed problem with lint-staged
Mil4n0r a03fa67
Testing husky
Mil4n0r b133c2c
Installed husky correctly
Mil4n0r e3c39d4
Fixed tables in website documentation
Mil4n0r 37907b6
Updated package-lock
Mil4n0r 7d79cf4
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 56a8eb3
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 24833ad
Removed duplicate
Mil4n0r 5436b2c
Merge with master
Mil4n0r 343663c
Removed duplicate babel config
Mil4n0r 3659e8a
Merge branch 'master' into Mil4n0r/code_quality_monorepo
Mil4n0r 41fabc6
Fixed eslint configuration
Mil4n0r 1ccaf08
Reactivated husky
Mil4n0r 9f1de1f
Removed duplicate file
Mil4n0r 31ea535
Fixed typo
Mil4n0r 60758ec
Removed prettier from lint-staged flow to make review easier
Mil4n0r cf14339
Merge branch 'master' into Mil4n0r/code_quality_monorepo
Mil4n0r 5e7b4bb
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 8a67d40
Merge branch 'Mil4n0r/code_quality_monorepo' of github.com:dxc-techno…
Mil4n0r bcda53d
Merge branch 'master' into Mil4n0r/code_quality_monorepo
Mil4n0r faf4e5b
Merge branch 'master' into Mil4n0r/code_quality_monorepo
Mil4n0r 6a61e76
Removed unneeded lines in tests
Mil4n0r fb2f22f
Added additional rule for .storybook eslint
Mil4n0r 6c99e45
Added additional rule for .storybook eslint
Mil4n0r 71bde24
Updated TS configuration
Mil4n0r 17f3e63
Test
Mil4n0r 3135dee
Test husky
Mil4n0r 8096ba0
Test include
Mil4n0r 807fa29
Restore husky settings
Mil4n0r c9caa64
Fixed most of DTS build errors
Mil4n0r 83d6ef9
Fixed more DTS build errors
Mil4n0r 6b79326
Solved conflicts and added most of the remaining types
Mil4n0r b4ecbd5
Removed deprecated lines from husky config
Mil4n0r c9dca5f
Fixed package-lock
Mil4n0r b75b359
Fixed remaining types from lib
Mil4n0r c6f83bf
Removed old TODO's
Mil4n0r 3d1e8bb
Finished fixing the types of the project
Mil4n0r da9a889
Fixed visual bug for Slider
Mil4n0r 3271892
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r d0ea1b8
Fixed eslint for scripts
Mil4n0r 977cbb0
Fixed warning in accordion
Mil4n0r 4dd6f98
Added missing parenthesis in accordion
Mil4n0r b76afb2
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 4c99aff
Fixed typings
Mil4n0r 127ab0a
Merge branch 'master' into Mil4n0r/code_quality_monorepo
Mil4n0r ec86ec1
Added tsc to check typescript errors
Mil4n0r 8a5038e
Merge branch 'Mil4n0r/code_quality_monorepo' of github.com:dxc-techno…
Mil4n0r b4bd635
Fixed typings problems for new tabs structure
Mil4n0r 0b2198c
Fixed tests not passing for new tabs structure
Mil4n0r ce8a1f5
Fixed minor problems in tabs display
Mil4n0r e8a308f
Merge branches 'Mil4n0r/code_quality_monorepo' and 'master' of github…
Mil4n0r c745a02
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r f720ae2
Fixed some minor problems in Toast behavior
Mil4n0r 12b4c59
Fixed extension of test helper
Mil4n0r 8949350
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r c9118fd
Resolved conflicts
Mil4n0r a37ca53
Fixed build problems
Mil4n0r aee09b3
Fixed typos
Mil4n0r 4cb3a9b
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 72092a0
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 2e89de0
Merge branch 'master' into Mil4n0r/code_quality_monorepo
Mil4n0r 7ef6ff7
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
4f19395
Fixed problems for build
d59da8d
Restored package-lock
0ba3607
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
Mil4n0r 8fbe712
Removed unused import
Mil4n0r 1f6201c
Fixed build problems
Mil4n0r 4089918
Fixed FocusLock bug
Mil4n0r dbe0c14
Merge branch 'Mil4n0r/strict-mode-true' of github.com:dxc-technology/…
Mil4n0r 618156e
Fixed build problems and most of the remaining types
Mil4n0r 48c579b
Removed some dangerous type assertions to use type guards instead
Mil4n0r 6b98af6
Fixed package-lock
Mil4n0r 29d90c7
Merge branch 'Mil4n0r/strict-mode-true' of github.com:dxc-technology/…
Mil4n0r f09637c
Merge branch 'Mil4n0r/strict-mode-true' of github.com:dxc-technology/…
Mil4n0r ee87b51
Merge branch 'master' into Mil4n0r/code_quality_monorepo
Mil4n0r 5f6092c
Removed more type assertions
Mil4n0r 8fc1b12
Merge branch 'Mil4n0r/code_quality_monorepo' of github.com:dxc-techno…
Mil4n0r 8002907
Solved build problem
Mil4n0r 22b4ae4
Restored popover import
Mil4n0r bcb9f90
Restored types for select
Mil4n0r 9a2d297
Removed redundant return undefined
55b4b36
Updated FocusLock with latest changes
e7d00ec
Fixed minor issues
6d375a1
Removed unwanted TODO
278941c
Added removed return
108d8e2
Merge branch 'master' of github.com:dxc-technology/halstack-react int…
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Empty file.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
npx lint-staged |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
module.exports = { | ||
"(apps|packages)/**/*.(ts|tsx|js)": (filenames) => { | ||
return [ | ||
`npx eslint --fix --max-warnings 0 ${filenames.map((file) => `"${file}"`).join(" ")}`, | ||
// `npx prettier --write ${filenames.map((file) => `"${file}"`).join(" ")}` | ||
]; | ||
}, | ||
}; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1 @@ | ||
http://localhost | ||
%25PUBLIC_URL%25 | ||
http://localhost |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,19 @@ | ||
{ | ||
"extends": "next/core-web-vitals" | ||
"extends": [ | ||
"next/core-web-vitals", | ||
"@dxc-technology/eslint-config/react-internal.js", | ||
"@dxc-technology/eslint-config/next.js" | ||
], | ||
"rules": { | ||
"no-unused-vars": [ | ||
"warn", | ||
{ | ||
"argsIgnorePattern": "^_" | ||
} | ||
], | ||
// TODO: Remove rules | ||
"react/no-unescaped-entities": "off", | ||
"import/no-anonymous-default-export": "off", | ||
"react-hooks/exhaustive-deps": "off" | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The prettier part will be uncommented on a separate PR to prevent increasing the review size (even further)