1ecdf4cd56
Bring examples section up to date, and save it from obvious bitrot
2018-01-24 13:26:03 +00:00
4c8f148c95
Distinguishing Product Reviews isn't helpful.
...
(Feedback from Ekr)
2018-01-24 13:26:03 +00:00
955697eb40
Remove comments about RACI
...
(Feedback from Ekr)
2018-01-24 13:26:03 +00:00
6c45571497
Add comment about smaller reviews
2018-01-24 13:26:03 +00:00
d87541918e
Merge pull request #54 from joewalker/issue-template
...
Add issue template
2018-01-23 17:20:47 +00:00
d7cffb5e4c
Add issue template
2018-01-23 11:31:48 +00:00
bc552cd82e
Merge pull request #45 from joewalker/fix-homepage-links
...
Make newsletter links work on Github project home page
2018-01-02 16:46:14 +00:00
b1ee4bed68
Add Newsletter 5 and fix forum links
2018-01-02 16:43:56 +00:00
5cb76f1ea0
Merge branch 'master' into fix-homepage-links
2018-01-02 16:37:12 +00:00
4e3a74ff28
Merge pull request #46 from mozilla/mykmelez-patch-1
...
fix indentation of list hierarchies
2018-01-02 16:21:55 +00:00
c5a18e4c85
Merge pull request #50 from mykmelez/fluent-in-prefs
...
Lightweight design review of Fluent in Preferences
2018-01-02 16:19:44 +00:00
7559af8c1e
Merge pull request #47 from Mossop/newsletter5
...
Add newsletter 5
2018-01-02 16:18:05 +00:00
1840dd3d22
lightweight design review of Fluent in prefs
2017-12-19 15:07:12 -08:00
c8ed8e4a5b
Adding links to the Firefox Data Stores documentation from Browser Ar… ( #11 )
...
* Adding links to the Firefox Data Stores documentation from Browser Arch repo
* Address review comments @jwalker
* Address review comments @rnewman
* Address review comments @rnewman
* Update filename to 007-firefox-data-stores.md
* Update to 0010
2017-12-15 12:22:51 -06:00
7b8bca1a57
Merge pull request #49 from Mossop/typesafety
...
Add type safety system information.
2017-12-06 12:47:34 +00:00
2bd44de739
Add type safety system information.
2017-12-05 14:15:27 -08:00
6872ca43ee
Merge pull request #48 from comex/patch-1
...
Grammar fix
2017-12-01 11:34:41 -08:00
a2b063a0b8
Grammar fix
2017-11-29 22:37:46 -08:00
1efbd06cd0
Add newsletter 5
2017-11-29 13:03:04 -08:00
1b872b095a
fix indentation of list hierarchies
2017-11-21 14:21:03 -08:00
7b356ff5bd
Archive Sync & Storage roadmap review document ( #43 ) r=joewalker
...
On 2017-11-09 we reviewed a proposal to address syncing and storage within Firefox.
This is a straightforward Markdownification of the review document, with footnotes flattened into links.
2017-11-21 18:54:37 +00:00
21d53f344a
Make newsletter links work on Github project home page
...
See https://github.com/mozilla/firefox-browser-architecture
Fixes #44
2017-11-21 11:23:42 +00:00
ac57559384
Doc updates and minor code sample changes ( #41 )
...
* Make the <project_path> consistent within examples
* Explicitly declare crate_type; clean up pointers.
2017-11-21 10:53:47 +00:00
e96e4b7fdb
Merge pull request #40 from joewalker/review-update
...
Updates to arch review doc following XBL removal review
2017-10-27 11:59:16 +01:00
711ee29328
Nit: to → the
...
Context: https://github.com/mozilla/firefox-browser-architecture/pull/40#discussion_r147216571
2017-10-27 11:57:58 +01:00
cccf3ea2e4
Clarify/simplify the roles of chair and reviewer
...
Context: https://github.com/mozilla/firefox-browser-architecture/pull/40#discussion_r146730034
2017-10-26 16:14:18 +01:00
36ceb71af3
A couple of clarifications from Myk
...
See https://github.com/mozilla/firefox-browser-architecture/pull/40#discussion_r146727418
for discussion.
2017-10-26 15:52:16 +01:00
55d3d1a575
Make the things identified by the brief clearer
2017-10-26 15:44:02 +01:00
d4798e3300
Minor tweak to the text of the above
2017-10-25 12:04:17 +01:00
3d318b3e18
Add stakeholders back in with better description
2017-10-25 11:50:19 +01:00
89ede148f3
Flesh out how to staff a meeting
2017-10-24 17:31:31 +01:00
3752242890
Add a suggested agenda (which indents the attendees section)
2017-10-24 15:00:18 +01:00
76bb1d4481
Use smart quotes
2017-10-24 14:59:45 +01:00
827200832a
Add an example of a successful review
2017-10-24 14:49:55 +01:00
5294a9ab2a
Add 'vision' to what a review packet should include.
2017-10-24 14:49:39 +01:00
ae54631149
Remove 'and the stakeholders'
...
The review needs to include stakeholders (defined elsewhere) but the
design packet doesn't need to specify this. They should know who they
are because they are at the review meeting.
2017-10-24 14:48:45 +01:00
f5e61ed094
Remove mention of an 'initial meeting'
...
It isn't defined below any more.
2017-10-24 14:47:09 +01:00
ee2dbc6a59
Define the new role of a 'chair'
...
(See previous commit for details)
2017-10-24 14:45:58 +01:00
40eff60b91
Rename 'chair' to 'reviewer'
...
The XBL review, and probably the Sync and Storage review will separate
the roles of the chair and reviewer. Renaming 'chair' to 'reviewer' is
step 1.
2017-10-24 14:44:28 +01:00
25efcc59f1
Add accessibility to 'forces on the system' ( #39 )
2017-10-23 09:48:50 -07:00
15022f73af
Fix direct post link.
2017-10-19 22:20:33 +00:00
1c5adfaac2
Add fourth newsletter.
2017-10-19 21:04:08 +00:00
72ca6cb2a0
Add the XBL design review packet ( #38 )
2017-10-13 10:31:47 -07:00
4a623c3f0a
Fix typos.
2017-10-10 17:46:15 +00:00
95f4bddb76
Add blog post for Rust on Android ( #34 )
...
* Add blog post for Rust on Android
* update README with link to new post
* Address review comments @rnewman @jwalker
* Update post to make copying cargo-config.toml to the .cargo directory non-optional
* Address review comments @jwalker
2017-10-05 10:06:04 -04:00
c187ad4447
Merge pull request #35 from joewalker/review-proposal
...
Phrase the review as a proposal rather than a question
2017-09-29 13:37:58 +01:00
681da2b34a
Address rnewman review
2017-09-29 13:37:03 +01:00
2df5c94b43
Merge branch 'master' into review-proposal
2017-09-29 13:21:11 +01:00
41d3692d6d
Brief should identify review that has already happened.
...
I think this is helpful for identifying gaps. For example, if the team
notes that they've talked to two other teams about their design, that
might prompt some of the reviewers (or the team itself, as they write
the brief) to notice that there's a third team that they also need to
talk to.
2017-09-29 13:18:25 +01:00
d0a4aca4fe
Fix links to newsletters.
2017-09-25 13:08:54 -07:00
Joe Walker
Myk Melez
Emily Toop
Dave Townsend
comex
Richard Newman
Grisha Kruglov
Brian Grinstead
L. David Baron