Bug 1567124 - remove useless templates from beetmoverscript as we switched to DA

mozilla-releng / beetmoverscript

An implementation of scriptworker for moving TC artifacts to all release-related buckets.

1 stars 19 forks source link

Bug 1567124 - remove useless templates from beetmoverscript as we switched to DA #239

Closed MihaiTabara closed 5 years ago

MihaiTabara commented 5 years ago

@JohanLorenzo Just to double-check, we have declarative artifacts across all trees, right?

I'm doing a more serious cleanup in https://github.com/mozilla-releng/beetmoverscript/pull/233/ but I'm still blocked there by Devedition and some other stuff that's currently riding the trains. But these templates are un-used AFAIK so at least we can remove them for now. Cleanup baby steps :)

MihaiTabara commented 5 years ago

Coverage is expected to decrease a bit since we're removing some of the tests. That's okay, we're going to remove the maven associated functions too soon.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 954

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.745%

Totals
Change from base Build 950:	0.0%
Covered Lines:	787
Relevant Lines:	797

💛 - Coveralls

MihaiTabara commented 5 years ago

Yeah, I confirm, esr60 since uses them. I'm still debating whether it makes sense to uplift patches or just to wait for its EOL in the falls. I'm adding support for Devedition too this week so hopefully there's going to be more templates to kill soon :)