Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
xSearch
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Wiki
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Snippets
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Package Registry
Container Registry
Model registry
Operate
Environments
Terraform modules
Monitor
Incidents
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Terms and privacy
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
1x Contrib Modules
xSearch
Merge requests
!15
dxp/issues#757: Add eslint and format the code
Code
Review changes
Check out branch
Download
Patches
Plain diff
Merged
dxp/issues#757: Add eslint and format the code
dxp/issues#757-linting
into
develop
Overview
1
Commits
1
Pipelines
0
Changes
29
Merged
Hossein Rashno
requested to merge
dxp/issues#757-linting
into
develop
2 years ago
Overview
1
Commits
1
Pipelines
0
Changes
29
Expand
Test:
Run
npm run lint
and make sure there is no error and the warnings there are fine to stay like that for now
Compile the app with TS and make sure there is no TS error
Run the app and make sure everything works as expected
0
0
Merge request reports
Compare
develop
develop (base)
and
latest version
latest version
8bf18c34
1 commit,
2 years ago
29 files
+
515
−
193
Inline
Compare changes
Side-by-side
Inline
Show whitespace changes
Show one file at a time
Files
29
Search (e.g. *.vue) (Ctrl+P)
src/components/factories/appFactory.tsx
+
2
−
0
Options
import
React
from
"
react
"
;
import
{
configService
}
from
"
@services
"
;
import
PortalFactory
from
"
./portalFactory
"
;
Loading