mgechev/codelyzer

bhaskerchari

@bhaskerchari

Hello @andichou09 ,

You can just simply remove the two rules no-access-missing-member &
templates-use-public from your ts-lint.json since these are no more supported in latest version of Codelyzer and so are deprecated. To get the behaviour for the removed rules we can take advantage from Angular Language Service vscode extension which is even documented in official core angular doc. You probably also need to consider relooking at some of the other rules in your tslint.json which may not be supported.

paulshelestovich

@paulshelestovich

hi, seems like this one is still reproducible on the latest version - mgechev/codelyzer#64

I've created a simple example here http://codelyzer.com/:

@Component({
  selector: 'sg-parent',
  template: '<h1>Parent</h1>'
})
class ParentComponent implements OnInit {
  public hero: Hero;

  public ngOnInit() {
    console.log('Parent');
  }
}

@Component({
  selector: 'sg-child',
  template: '<h1>Child</h1>'
})
class ChildComponent extends ParentComponent {
  public ngOnInit() {
    super.ngOnInit();
    console.log('Child');
  }
}

paulshelestovich

@paulshelestovich

@mgechev is it known issue?

sam-s4s

@sam-s4s

Is there a rule to check for components without selectors?

paulshelestovich

@paulshelestovich

is this thread alive?

sam-s4s

@sam-s4s

it's pretty quiet :P

jogelin

@jogelin

Hello, I configured a repo which contains multiple angular cli projects and I would like to reuse a tslint but it seems that I have rule that check if I have an .angular-cli.json at the root of my project : ENOENT: no such file or directory, open 'C:\Users\jgelin\dev\projects\workspaces.angular-cli.json'
Someone knows how can I disable this check ?

Ok it seems it's a rule from Nx...

SRP-the-ceder

@SRP-the-ceder

anyone know of any tool that will analyse performance of my Angular 4 code

mertdeg2

@mertdeg2

I am looking to upgrade codelyzer TS dependency to 2.7.2 from 2.4

But i am super noob at this

how do i run codelyzer tests?

@mgechev

luboslav

@luboslav

Hi, is codelyzer aimed primary on rules that are related to angular or could it be used for checking general typescript coding conventions? For example https://angular.io/guide/styleguide#style-03-04

Minko Gechev

@mgechev

@luboslav we're discussing removal of the coding practices from the style guide.

paulshelestovich

@paulshelestovich

@mgechev have you seen the example I've posted above? are there any plans to fix the rules with inheritance? thanks

Sam Grönblom

@sgronblo

Hello, I just set up a new project using ng new for testing, but when running ng lint no errors seem to be shown for my template files. What could be wrong?

sam-s4s

@sam-s4s

I'm pretty sure the linting process only covers your .ts files - but not the templates

Sam Grönblom

@sgronblo

Well, I found some pages that suggest otherwise: angular/angular-cli#4351

Rob

@borriej

hi all

im having a issue

with angular-whitespace in component.ts

mgechev/codelyzer#692

someblue

@someblue

hi all.

someblue

@someblue

I write a deliberate wrong test case, run test by 'npm run test'. But Chai pass the wrong case.
I find the reason is that assertFailure in test/testHelper.ts catch the exception and don't continue to throw it.
Did I run the test in wrong way? Thanks for any help.

tejaspanchal2011

@tejaspanchal2011

Hello

tejaspanchal2011

@tejaspanchal2011

is there any way to generate and export error report in codelyzer?

@mgechev

Paul King

@King-i

hi all, great project! Trying to create a custom rule to validate element types a directive is bound to, for the life of me i cannot figure out how to get the parent element type from BoundDirectivePropertyAst anyone know of a way todo this? thanks in advance!

Eduard Paprotski

@EdwOK

Hi guys, Anybody faced with the that the i18n rule works incorrectly in cases like here: <span>...</span> for example ?

JaveedShaik

@JaveedShaik

hi @mgechev

I am having angular 6 application,How to integrate codelyzer in it

Please let me know the steps for it

Thanks

getting errors below :

The specified path does not exist: 'D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\e2e\tsconfig.e2e.json'.

FatalError: error TS5058: The specified path does not exist: 'D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\e2e\tsconfig.e2e.json'.

at new FatalError (D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\tslint\lib\error.js:27:28)
at Function.Linter.createProgram (D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\tslint\lib\linter.js:55:19)
at Observable.rxjs_1.Observable.obs [as _subscribe] (D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\@angular-devkit\build-angular\src\tslint\index.js:62:44)
at Observable._trySubscribe (D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\rxjs\internal\Observable.js:43:25)
at Observable.subscribe (D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\rxjs\internal\Observable.js:29:22)
at D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\rxjs\internal\util\subscribeTo.js:22:31
at Object.subscribeToResult (D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\rxjs\internal\util\subscribeToResult.js:7:45)
at MergeMapSubscriber._innerSub (D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\rxjs\internal\operators\mergeMap.js:75:38)
at MergeMapSubscriber._tryNext (D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\rxjs\internal\operators\mergeMap.js:72:14)
at MergeMapSubscriber._next (D:\javeed\Projects\RandDProjects\EMS - V3 Research\EMS.Client\node_modules\rxjs\internal\operators\mergeMap.js:55:18)

JaveedShaik

@JaveedShaik

I am using angular 6 in my project

Will Codelyzer supports it ?

JaveedShaik

@JaveedShaik

@tarlepp

Zama Khan Mohammed

@mohammedzamakhan

Hi Guys, whats the best way to contribute to codelyzer, when developing a new rule, how do you develop it, do you run test:watch, and how exactly do you only watch the rule you are working on?

I do not see any steps in the contributing.md file

123aakash

@123aakash

Hi guys I want to order my methods so that lifecycle methods remain together

is there any way to do that

Michael Frank

@MFrank84

Hi :)

I recently updated codelyzer to version 5.0.1 and added the 'no-inputs-metadata-property" rule. But now my IDE (IntelliJ) is throwing the following error

TypeError: Cannot read property 'Input' of undefined
    at /workspace/terra-components/node_modules/codelyzer/noInputsMetadataPropertyRule.js:38:798
    at Object.<anonymous> (/workspace/terra-components/node_modules/codelyzer/noInputsMetadataPropertyRule.js:45:2)
    at Module._compile (internal/modules/cjs/loader.js:799:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:810:10)
    at Module.load (internal/modules/cjs/loader.js:666:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:606:12)
    at Function.Module._load (internal/modules/cjs/loader.js:598:3)
    at Module.require (internal/modules/cjs/loader.js:705:19)
    at require (internal/modules/cjs/helpers.js:14:16)
    at loadRule (/usr/local/lib/node_modules/tslint/lib/ruleLoader.js:102:12)
Process finished with exit code -1

Am I doing something wrong?

Eduard Paprotski

@EdwOK

@mgechev Hi, maybe do you have any plans to add ngx-translate linter rules?

Akash Pius

@akashpius

@mgechev Hi, is Angular 7 and is suported by codelyzer

@mgechev will codelyzer supports to angular 7 and 8 versions

Where communities thrive

People

Repo info

Activity