Merge lp:~aacid/unity8/coding_update into lp:unity8
| Status: | Rejected |
|---|---|
| Rejected by: | Albert Astals Cid on 2015-11-09 |
| Proposed branch: | lp:~aacid/unity8/coding_update |
| Merge into: | lp:unity8 |
| Diff against target: |
138 lines (+3/-125) 1 file modified
CODING (+3/-125) |
| To merge this branch: | bzr merge lp:~aacid/unity8/coding_update |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-11-05 | |
| Daniel d'Andrada (community) | 2015-11-04 | Disapprove on 2015-11-04 | |
|
Review via email:
|
|||
Commit Message
Point to the web
Let's not maintain two copies of the same thing that will get old and unsynced at different stages
Description of the Change
* Are there any related MPs required for this MP to build/function as expected?
No
* Did you perform an exploratory manual test run of your code change and any related functionality?
N/A
* Did you make sure that your branch does not contain spurious tags?
Yes
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A
* If you changed the UI, has there been a design review?
N/A
| Michael Zanetti (mzanetti) wrote : | # |
I'm in favor of removing redundancy. Whether it is removing them here or generate the website out of it, I don't have a strong opinion. Generating the website out of it would be definitely more work tho.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2022
http://
Executed test runs:
UNSTABLE: http://
FAILURE: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:2022
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Ok, it seems it didn't get enough consensus, someone else have to fix this duplication then.
Unmerged revisions
- 2022. By Albert Astals Cid on 2015-11-04
-
Point to the web
Let's not maintain two copies that will get old and unsynced at different stages

I prefer to keep those as close to the code as possible. They are more prone to getting outdated if kept separately.
I would rather do the other way around and build the web page from the this text file, like online documentation of libs.