Skip to content
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

Disable lazy loading on my content #1021

Conversation

newswangerd
Copy link
Member

backport: #922

akaRem and others added 2 commits August 3, 2018 14:11
Currently there are a lot of unused variables in project and they are
mostly represented as unused imports. There are also some unused
variables in scope of module and function definitions as well as some
unused private fields of TS classes.

This patch enables no-unused-variable rule in tslint config.
Due to the fact that it's not possible to check only unused imports,
other rule warnings must be fixed too.
So this patch includes removal of all unused variables found in
function bodies and variables declared but unused in scope of module.

All unused constructor args are removed too since Angular dependency
injection is not dependent on argument order.

Signed-off-by: Ivan Remizov <iremizov@gmail.com>
(cherry picked from commit 22206bf)
(cherry picked from commit f39d0c1)
@newswangerd newswangerd added the backport Backport pull request label Aug 3, 2018
@newswangerd newswangerd merged commit 6817359 into ansible:release/3.0.6 Aug 3, 2018
@newswangerd newswangerd deleted the backport/922-lazy-load-my-content branch August 23, 2018 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Backport pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants