Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Add linting rule for adding a license #3880

Closed
sblatz opened this issue Jul 3, 2019 · 13 comments
Closed

Add linting rule for adding a license #3880

sblatz opened this issue Jul 3, 2019 · 13 comments
Labels
eng:health Improve code health Hacktoberfest Highlights issues for Hacktoberfest contributors to help us! help wanted Help wanted from a contributor. More complex than good first issue. P5 Never, but will accept a patch

Comments

@sblatz
Copy link
Contributor

sblatz commented Jul 3, 2019

We should have a linting rule that triggers if a new file does not include the Mozilla license.

┆Issue is synchronized with this Jira Task

@sblatz sblatz added the eng:health Improve code health label Jul 3, 2019
@Amejia481
Copy link
Contributor

Similar issue on AC -> mozilla-mobile/android-components#846

@sblatz sblatz added the P5 Never, but will accept a patch label Jul 20, 2019
@sblatz
Copy link
Contributor Author

sblatz commented Jul 20, 2019

While I'd love to have this, it's not the top of our priorities. Would love a contributor PR on this!

@NotWoods NotWoods added the help wanted Help wanted from a contributor. More complex than good first issue. label Jul 22, 2019
@dector
Copy link
Contributor

dector commented Sep 22, 2019

Is header format defined in mozilla-mobile/android-components#4460 works in this project as well?

@NotWoods
Copy link
Contributor

Yes, the same header format works here too 😄

@dector
Copy link
Contributor

dector commented Sep 23, 2019

@NotWoods Cool. I'll make PR today.

@dector
Copy link
Contributor

dector commented Sep 23, 2019

Should we publish tooling-detekt from AC or just copy-past license plugin? 😄

@pocmo
Copy link
Contributor

pocmo commented Sep 24, 2019

@dector Yes, can you try publishing and consuming it from mavenLocal()?

@dector
Copy link
Contributor

dector commented Oct 1, 2019

@pocmo Sorry for the late reply. I was busy a bit. :(

If I got your proposal right - the idea is to publish module from AC project to local maven repository and reuse it in Fenix project? Isn't it adding explicit dependency on AC?

@pocmo
Copy link
Contributor

pocmo commented Oct 1, 2019

@dector Oh, I meant only for testing if everything works. After that we'd actually publish it from AC to maven.mozilla.org

@dector
Copy link
Contributor

dector commented Oct 1, 2019

@pocmo Ah, sure! Using rulesets from a maven repository works similarly to using rulesets from a separate module.

Should I just set publish: true in .buildconfig.yml or there are is another (preferable) way to publish this ruleset for internal use?

@pocmo
Copy link
Contributor

pocmo commented Oct 1, 2019

@pocmo Ah, sure! Using rulesets from a maven repository works similarly to using rulesets from a separate module.

Yeah, it should. But I am not sure whether our build/publish mechanism will work as expected. That's why I would love to try that with a local repository before enabling it in buildconfig.yml. If that works then yes, let's set it to publish: true and then test again with the snapshots. :)

@dector
Copy link
Contributor

dector commented Oct 1, 2019

@pocmo Awesome! I tested ./gradlew :tooling-detekt:publishToMavenLocal with maven-publish plugin.

I guess main publishing mechanism is implemented in automation/taskcluster/decision_task.py. If it's needed I can try to dig into it and try to publish components to local maven repository first. But I don't expect any troubles with it as other components are published to mozilla maven without any issue.

@severinrudie severinrudie added the Hacktoberfest Highlights issues for Hacktoberfest contributors to help us! label Oct 1, 2019
dector added a commit to dector/fenix that referenced this issue Oct 3, 2019
dector added a commit to dector/fenix that referenced this issue Oct 3, 2019
dector added a commit to dector/fenix that referenced this issue Apr 4, 2020
@eliserichards
Copy link
Contributor

Closing in favor of mozilla-mobile/android-components#6351

dector added a commit to dector/fenix that referenced this issue May 24, 2020
dector added a commit to dector/fenix that referenced this issue May 30, 2020
dector added a commit to dector/fenix that referenced this issue Jun 21, 2020
dector added a commit to dector/fenix that referenced this issue Jul 16, 2020
dector added a commit to dector/fenix that referenced this issue Jul 16, 2020
dector added a commit to dector/fenix that referenced this issue Jul 16, 2020
liuche added a commit that referenced this issue Jul 20, 2020
* #3880. Update detekt to 1.9.1

* #3880. Use `AbsentOrWrongFileLicense` detekt rule

* #3880. Update detekt baseline

Co-authored-by: Denys M <dector9@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
eng:health Improve code health Hacktoberfest Highlights issues for Hacktoberfest contributors to help us! help wanted Help wanted from a contributor. More complex than good first issue. P5 Never, but will accept a patch
Projects
None yet
Development

No branches or pull requests

7 participants