Jetpack/Weekly Meeting/2014-08-19: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
(Undo revision 1009445 by Zombie (talk))
 
(7 intermediate revisions by 3 users not shown)
Line 4: Line 4:
* High Priority Work
* High Priority Work
** [https://bugzilla.mozilla.org/show_bug.cgi?id=915376 Native Jetpack Support]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=915376 Native Jetpack Support]
*** [https://bugzilla.mozilla.org/show_bug.cgi?id=915376 Bug 915376] Updating the Native Jetpack JEP
*** [https://bugzilla.mozilla.org/show_bug.cgi?id=1040270 Bug 1040270] If an add-on has a ./package.json file, then call it a jetpack
*** [https://bugzilla.mozilla.org/show_bug.cgi?id=1040238 Bug 1040238] Implementing loadManifestFromJSON
** [https://bugzilla.mozilla.org/show_bug.cgi?id=961191 JPM]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=961191 JPM]
*** [https://github.com/mozilla/jpm/pull/142 Pull Request 142] Don't overwrite existing bootstrap.js and install.rdf, instead use them in the xpi
*** Updated version to 0.0.13
** [https://gist.github.com/Gozala/9490234 Devtools SDK]
** [https://gist.github.com/Gozala/9490234 Devtools SDK]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=994280 Button badges]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=994280 Button badges]
Line 11: Line 16:
* SDK
* SDK
** Blockers?
** Blockers?
* e10s updates <span id=zombie>(zombie)</span>
** emitSync is completely dead in e10s, [https://bugzilla.mozilla.org/show_bug.cgi?id=1042587 bug 1042587]
*** awful timing.. :(
** plus after lots of test fixes and disabling, we are finally green :\
* Roundtable
* Roundtable
** [https://bugzilla.mozilla.org/show_bug.cgi?id=998277 Uncaught Promise rejections will now throw in tests]
** [https://bugzilla.mozilla.org/show_bug.cgi?id=998277 Uncaught Promise rejections will now throw in tests]
** [https://groups.google.com/forum/#!topic/mozilla-labs-jetpack/0BsJ1vf9udI Most Voted on SDK Bugs] from Ben McCann
** [https://groups.google.com/forum/#!topic/mozilla-labs-jetpack/0BsJ1vf9udI Most Voted on SDK Bugs] from Ben McCann
** new contributors shoutout (to be released as part of firefox 34)
*** [https://github.com/riteshmahato Ritesh Mahato] reduced some useless math
*** [https://github.com/poiru Birunthan Mohanathas] fixed some debug build warnings
*** [https://github.com/spite Jaume Sanchez] ''foxed'' a typo in readme instuctions
*** [https://github.com/kewisch Philipp Kewisch] added ability to use custom bootstrap/install.rdf with jpm


= Bugs =
= Bugs =
Line 35: Line 49:


= Attendees =  
= Attendees =  
* zombie
* Mossop
* Jeff
* Jordan
* Erik
* Will
* Matteo


= Minutes =
= Minutes =
==High Priority Work==
===[https://bugzilla.mozilla.org/show_bug.cgi?id=915376 Native Jetpack Support]===
Erik: not much to report. Working on updating jep, has patch for Bug 1040270.
* [https://bugzilla.mozilla.org/show_bug.cgi?id=915376 Bug 915376] Updating the Native Jetpack JEP
* [https://bugzilla.mozilla.org/show_bug.cgi?id=1040270 Bug 1040270] If an add-on has a ./package.json file, then call it a jetpack
* [https://bugzilla.mozilla.org/show_bug.cgi?id=1040238 Bug 1040238] Implementing loadManifestFromJSON
===[https://bugzilla.mozilla.org/show_bug.cgi?id=961191 JPM]===
* [https://github.com/mozilla/jpm/pull/142 Pull Request 142] Don't overwrite existing bootstrap.js and install.rdf, instead use them in the xpi
* Updated version to 0.0.13
===[https://gist.github.com/Gozala/9490234 Devtools SDK]===
No Irakli, no update.
===[https://bugzilla.mozilla.org/show_bug.cgi?id=994280 Button badges]===
Matteo: attached a patch for platform bug, waiting for review. In bug 994280, in the screenshot, badges are misaligned, or look misaligned. This is a platform-specific issue. It can go out as it is, and be fixed later in platform. Only happens when the icon is smaller than the proper size. Can be an FAQ item then. Let's land it as it is, fix in the platform if it becomes an issue for many people.
===[https://bugzilla.mozilla.org/show_bug.cgi?id=1035512 Mochitestification]===
Waiting for 1 review, then test and addon test stuff can land. This week, we hope.
===[https://bugzilla.mozilla.org/show_bug.cgi?id=1035970 SDK 1.17]===
SDK 1.17 is failing some tests. Promise rejections in sidebar. But that's not actually what we are shipping (modules are in Firefox). Ship anyway. zombie: we should test behaviour when a module does not exist in the SDK but does exist in Firefox.
==SDK==
===Blockers?===


=== High Priority Work ===
===e10s updates <span id=zombie>(zombie)</span>===
* emitSync is completely dead in e10s, [https://bugzilla.mozilla.org/show_bug.cgi?id=1042587 bug 1042587]. Awful timing. Have to redo all zombie's work. :(


=== Blockers ===
== Roundtable==
* [https://bugzilla.mozilla.org/show_bug.cgi?id=998277 Uncaught Promise rejections will now throw in tests]
* [https://groups.google.com/forum/#!topic/mozilla-labs-jetpack/0BsJ1vf9udI Most Voted on SDK Bugs] from Ben McCann: Bugzilla votes may not be the most reliable way to gauge the importance of bugs, many people don't know about voting. Still, really happy that Ben got these bugs fixed.
* new contributors shoutout (to be released as part of firefox 34)
** [https://github.com/riteshmahato Ritesh Mahato] reduced some useless math
** [https://github.com/poiru Birunthan Mohanathas] fixed some debug build warnings
** [https://github.com/spite Jaume Sanchez] ''foxed'' a typo in readme instuctions
** [https://github.com/kewisch Philipp Kewisch] added ability to use custom bootstrap/install.rdf with jpm


=== Roundtable ===
* how should we acknowledge new contributors? Publish these meeting notes to planet? Add to about:credits? Friends of the Tree? Jeff will think about that.

Latest revision as of 17:08, 26 August 2014


Agenda

Bugs

Triage Followups

No results.

0 Total; 0 Open (0%); 0 Resolved (0%); 0 Verified (0%);


Open Bugs

Bugzilla query error

GD extension must be loaded.1


Attendees

  • zombie
  • Mossop
  • Jeff
  • Jordan
  • Erik
  • Will
  • Matteo

Minutes

High Priority Work

Native Jetpack Support

Erik: not much to report. Working on updating jep, has patch for Bug 1040270.

  • Bug 915376 Updating the Native Jetpack JEP
  • Bug 1040270 If an add-on has a ./package.json file, then call it a jetpack
  • Bug 1040238 Implementing loadManifestFromJSON

JPM

  • Pull Request 142 Don't overwrite existing bootstrap.js and install.rdf, instead use them in the xpi
  • Updated version to 0.0.13

Devtools SDK

No Irakli, no update.

Button badges

Matteo: attached a patch for platform bug, waiting for review. In bug 994280, in the screenshot, badges are misaligned, or look misaligned. This is a platform-specific issue. It can go out as it is, and be fixed later in platform. Only happens when the icon is smaller than the proper size. Can be an FAQ item then. Let's land it as it is, fix in the platform if it becomes an issue for many people.

Mochitestification

Waiting for 1 review, then test and addon test stuff can land. This week, we hope.

SDK 1.17

SDK 1.17 is failing some tests. Promise rejections in sidebar. But that's not actually what we are shipping (modules are in Firefox). Ship anyway. zombie: we should test behaviour when a module does not exist in the SDK but does exist in Firefox.

SDK

Blockers?

e10s updates (zombie)

  • emitSync is completely dead in e10s, bug 1042587. Awful timing. Have to redo all zombie's work. :(

Roundtable

  • how should we acknowledge new contributors? Publish these meeting notes to planet? Add to about:credits? Friends of the Tree? Jeff will think about that.