Closed Bug 856968 Opened 7 years ago Closed 6 years ago

cfx xpi should not include the tests folder

Categories

(Add-on SDK Graveyard :: General, defect, P3)

x86
All
defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: evold, Assigned: riadh.chtara)

References

Details

(Whiteboard: [good first bug])

Attachments

(1 file)

An AMO reviewer pointed out to me that the latest version of my no-ga add-on had a empty 'tests' folder, the repo has a file in that folder which was not shipped with the add-on, but there is a empty 'tests' folder, which should not be included.
Whiteboard: [good first bug]
Assignee: nobody → riadh.chtara
Status: NEW → ASSIGNED
Once bug 915376 lands this will be a bug for jpm.

Also this patch probably needs updating and needs tests before we could land it.
Depends on: native-jetpack
Priority: P3 → --
Sorry we won't be releasing any new versions of cfx, jpm is the replacement https://www.npmjs.com/package/jpm

For more information about this feature in jpm see https://github.com/mozilla/jpm/issues/168
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
No longer depends on: native-jetpack
You need to log in before you can comment on or make changes to this bug.