Page MenuHomeDevCentral
Differential D997

Analyze The Japan Times
ClosedPublic
Actions

Authored by Thibaut120094 on Jun 16 2017, 21:26.
Tags
None
Referenced Files
F3762757: D997.diff
Thu, Nov 21, 14:50
F3762756: D997.diff
Thu, Nov 21, 14:50
Unknown Object (File)
Tue, Nov 19, 09:08
Unknown Object (File)
Mon, Nov 18, 21:47
Unknown Object (File)
Mon, Nov 18, 07:57
Unknown Object (File)
Sun, Nov 17, 23:20
Unknown Object (File)
Sun, Nov 17, 20:02
Unknown Object (File)
Sat, Nov 16, 07:58
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rSTGb2d03e1ca957: Analyze The Japan Times
Summary

Analyze The Japan Times

Test Plan

http://www.japantimes.co.jp/news/2014/04/07/national/politics-diplomacy/kyoto-re-elects-yamada-to-top-post/

Diff Detail

Repository
rSTG Source templates generator
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
arcpatch-D997
Build Status
Buildable 1567
Build 1815: arc lint + arc unit

Event Timeline

Thibaut120094 created this revision.Jun 16 2017, 21:26
dereckson requested changes to this revision.Jun 17 2017, 23:37

Looks good.

Note pages/index.dat chmod shouldn't change, as there isn't rationale to have an executable there

pages/thejapantimes.php
26

Whitespace issue.

Ident with 4 spaces (we need an automatic check for arc diff by the way for this one, ideally with an autofix offered).

This revision now requires changes to proceed.Jun 17 2017, 23:37
Thibaut120094 updated this revision to Diff 2548.Jul 5 2017, 17:21
Thibaut120094 edited edge metadata.

Fixed whitespace issues and chmod change.

Thibaut120094 marked an inline comment as done.Jul 5 2017, 17:47
Harbormaster completed remote builds in B1556: Diff 2548.Jul 5 2017, 23:15
dereckson added a comment.Jul 5 2017, 23:17

Looks good to me.

dereckson requested changes to this revision.Jul 29 2017, 10:20
dereckson added inline comments.
pages/thejapantimes.php
25

Strict Standards: Only variables should be passed by reference in /usr/home/dereckson/dev/nasqueron/tools/3rdparty/source-templates-generator/pages/thejapantimes.php on line 24

This revision now requires changes to proceed.Jul 29 2017, 10:20
dereckson updated this revision to Diff 2566.Jul 30 2017, 12:50
dereckson edited edge metadata.

Use between for title

dereckson updated this revision to Diff 2567.Jul 30 2017, 12:51

Grammar

dereckson accepted this revision.Jul 30 2017, 12:52
This revision is now accepted and ready to land.Jul 30 2017, 12:52
Closed by commit rSTGb2d03e1ca957: Analyze The Japan Times (authored by Thibaut120094, committed by dereckson). · Explain WhyJul 30 2017, 12:52
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
pages/
1 line
26 lines

Diff 2566

View Options

pages/index.dat

Loading...
View Options

pages/thejapantimes.php

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator