1590037 - mozlint outputs absolute paths for source-test-mozlint-license

Status: NEW

(Developer Infrastructure :: Lint and Formatting, task)

Description

[Bastien Abadie [:bastien]]

As Bug 1568484 removed absolute paths output from mozlint, the licence checker should also output relative paths.

But a recent licence check task found an issue and reported it using the full path

Comment 1

[Sumagna Das]

(In reply to Bastien Abadie [:bastien] from comment #0)

As Bug 1568484 removed absolute paths output from mozlint, the licence checker should also output relative paths.

But a recent licence check task found an issue and reported it using the full path

can you please attach a file with this comment which contains the output (or whatever the links contained) as the links are not working (atleast not for me).

Comment 2

[Nirmay Dhruv [:nirmay]]

(In reply to Bastien Abadie [:bastien] from comment #0)

As Bug 1568484 removed absolute paths output from mozlint, the licence checker should also output relative paths.

But a recent licence check task found an issue and reported it using the full path

[:bastien], can you provide the links again as the previous ones don't seem to work? Also, I would like to figure something out and give it a try, so any other information regarding the bug will be useful!

Comment 3

[Nirmay Dhruv [:nirmay]]

[:Sylvestre], I just realized Bastien Abadie's profile is disabled, and cannot request information from him.
Can you provide the links (as the ones given in the description don't work) and any other information useful to start over the bug and I can give it a try?

Comment 4

[Sumagna Das]

(In reply to Nirmay Dhruv [:nirmay] from comment #3)

[:Sylvestre], I just realized Bastien Abadie's profile is disabled, and cannot request information from him.
Can you provide the links (as the ones given in the description don't work) and any other information useful to start over the bug and I can give it a try?

Thanks for pointing out that Bastien Abadie's account is disabled.

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Andi might be able to help. Sorry for the crazy latency!

Comment 6

[Andi [:andi]]

Please look here for the definition yml for the files that are excluded.
The implementation for the linter resides here.

Comment 7

[Riya]

Is this issue still open to working? I would like to work.

Comment 8

[sjbasak]

Hi, I am getting started with Open Source. I want to work on this issue. Would u assign this to me? And let me know how I start to work on it?

Comment 9

[Sylvestre Ledru [:Sylvestre]]

the bug will be assigned to you once you submitted the patch.
for the doc:
https://firefox-source-docs.mozilla.org/contributing/contribution_quickref.html
and comment #6 should be enough

Comment 10

[sjbasak]

Okay, I am getting started right away.

Comment 11

[sjbasak]

what build type should I choose while running bootstrap.py?