Single-player is proper English, single player and singleplayer are less correct.
(Contrary to multiplayer, which doesn't have a hyphen or space.)
Details
- Reviewers
Gallaecio - Commits
- rP23176: Single Player → single-player
Check for mistakes and omissions.
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 9512 Build 15880: Vulcan Build Jenkins Build 15879: Vulcan Build (Windows) Jenkins Build 15878: arc lint + arc unit
Event Timeline
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/267/display/redirect
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues: Executing section Source... Executing section JS... | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 192| 192| "tooltip": translate("Exits the game."), | 193| 193| "onPress": () => { | 194| 194| messageBox( | 195| |- 400, 200, | | 195|+ 400, 200, | 196| 196| translate("Are you sure you want to quit 0 A.D.?"), | 197| 197| translate("Confirmation"), | 198| 198| [translate("No"), translate("Yes")], | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 193| 193| "onPress": () => { | 194| 194| messageBox( | 195| 195| 400, 200, | 196| |- translate("Are you sure you want to quit 0 A.D.?"), | | 196|+ translate("Are you sure you want to quit 0 A.D.?"), | 197| 197| translate("Confirmation"), | 198| 198| [translate("No"), translate("Yes")], | 199| 199| [null, Engine.Exit]); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 194| 194| messageBox( | 195| 195| 400, 200, | 196| 196| translate("Are you sure you want to quit 0 A.D.?"), | 197| |- translate("Confirmation"), | | 197|+ translate("Confirmation"), | 198| 198| [translate("No"), translate("Yes")], | 199| 199| [null, Engine.Exit]); | 200| 200| } | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 195| 195| 400, 200, | 196| 196| translate("Are you sure you want to quit 0 A.D.?"), | 197| 197| translate("Confirmation"), | 198| |- [translate("No"), translate("Yes")], | | 198|+ [translate("No"), translate("Yes")], | 199| 199| [null, Engine.Exit]); | 200| 200| } | 201| 201| } | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 196| 196| translate("Are you sure you want to quit 0 A.D.?"), | 197| 197| translate("Confirmation"), | 198| 198| [translate("No"), translate("Yes")], | 199| |- [null, Engine.Exit]); | | 199|+ [null, Engine.Exit]); | 200| 200| } | 201| 201| } | 202| 202| ]; binaries/data/mods/public/gui/replaymenu/replay_filters.js | 188| » » switch·(sortKey) | | [NORMAL] ESLintBear (default-case): | | Expected a default case. binaries/data/mods/public/gui/session/menu.js | 469| » » button.onPress·=·(function(player,·stance)·{·return·function()·{ | | [NORMAL] ESLintBear (no-shadow): | | 'stance' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 501| » » button.onPress·=·(function(i,·resCode,·button)·{ | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 501| » » button.onPress·=·(function(i,·resCode,·button)·{ | | [NORMAL] ESLintBear (no-shadow): | | 'resCode' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 501| » » button.onPress·=·(function(i,·resCode,·button)·{ | | [NORMAL] ESLintBear (no-shadow): | | 'button' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 544| » button.onPress·=·(function(i)·{·return·function()·{ | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 610| » button.onPress·=·(function(i,·button)·{·return·function()·{ | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 610| » button.onPress·=·(function(i,·button)·{·return·function()·{ | | [NORMAL] ESLintBear (no-shadow): | | 'button' is already declared in the upper scope. Executing section cli...
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/779/display/redirect
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/268/display/redirect
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues: Executing section Source... Executing section JS... binaries/data/mods/public/gui/replaymenu/replay_filters.js | 188| » » switch·(sortKey) | | [NORMAL] ESLintBear (default-case): | | Expected a default case. binaries/data/mods/public/gui/session/menu.js | 469| » » button.onPress·=·(function(player,·stance)·{·return·function()·{ | | [NORMAL] ESLintBear (no-shadow): | | 'stance' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 501| » » button.onPress·=·(function(i,·resCode,·button)·{ | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 501| » » button.onPress·=·(function(i,·resCode,·button)·{ | | [NORMAL] ESLintBear (no-shadow): | | 'resCode' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 501| » » button.onPress·=·(function(i,·resCode,·button)·{ | | [NORMAL] ESLintBear (no-shadow): | | 'button' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 544| » button.onPress·=·(function(i)·{·return·function()·{ | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 610| » button.onPress·=·(function(i,·button)·{·return·function()·{ | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/menu.js | 610| » button.onPress·=·(function(i,·button)·{·return·function()·{ | | [NORMAL] ESLintBear (no-shadow): | | 'button' is already declared in the upper scope. | | [NORMAL] ESLintBear (indent): | | Expected indentation of 6 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | 392| 392| // Players see colors depending on diplomacy | 393| 393| g_DisplayedPlayerColors[i] = | 394| 394| g_ViewedPlayer == i ? getDiplomacyColor("self") : | 395| |- g_Players[g_ViewedPlayer].isAlly[i] ? getDiplomacyColor("ally") : | | 395|+ g_Players[g_ViewedPlayer].isAlly[i] ? getDiplomacyColor("ally") : | 396| 396| g_Players[g_ViewedPlayer].isNeutral[i] ? getDiplomacyColor("neutral") : | 397| 397| getDiplomacyColor("enemy"); | 398| 398| | | [NORMAL] ESLintBear (indent): | | Expected indentation of 7 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | 393| 393| g_DisplayedPlayerColors[i] = | 394| 394| g_ViewedPlayer == i ? getDiplomacyColor("self") : | 395| 395| g_Players[g_ViewedPlayer].isAlly[i] ? getDiplomacyColor("ally") : | 396| |- g_Players[g_ViewedPlayer].isNeutral[i] ? getDiplomacyColor("neutral") : | | 396|+ g_Players[g_ViewedPlayer].isNeutral[i] ? getDiplomacyColor("neutral") : | 397| 397| getDiplomacyColor("enemy"); | 398| 398| | 399| 399| g_DisplayedPlayerColors[0] = g_Players[0].color; | | [NORMAL] ESLintBear (indent): | | Expected indentation of 8 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | 394| 394| g_ViewedPlayer == i ? getDiplomacyColor("self") : | 395| 395| g_Players[g_ViewedPlayer].isAlly[i] ? getDiplomacyColor("ally") : | 396| 396| g_Players[g_ViewedPlayer].isNeutral[i] ? getDiplomacyColor("neutral") : | 397| |- getDiplomacyColor("enemy"); | | 397|+ getDiplomacyColor("enemy"); | 398| 398| | 399| 399| g_DisplayedPlayerColors[0] = g_Players[0].color; | 400| 400| } | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | 656| 656| "civ": setStringTags(g_CivData[g_Players[g_ViewedPlayer].civ].Name, { "font": "sans-bold-stroke-14" }), | 657| 657| "hotkey_civinfo": colorizeHotkey("%(hotkey)s", "civinfo"), | 658| 658| "hotkey_structree": colorizeHotkey("%(hotkey)s", "structree") | 659| |- }); | | 659|+ }); | 660| 660| } | 661| 661| | 662| 662| // Following gaia can be interesting on scripted maps | | [NORMAL] ESLintBear (indent): | | Expected indentation of 3 tabs but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/session.js |1211|1211| |1212|1212| let orderHotkeyTooltip = Object.keys(viewablePlayerStates).length <= 1 ? "" : |1213|1213| "\n" + sprintf(translate("%(order)s: %(hotkey)s to change order."), { |1214| |- "hotkey": setStringTags("\\[Click]", g_HotkeyTags), | |1214|+ "hotkey": setStringTags("\\[Click]", g_HotkeyTags), |1215|1215| "order": tooltipSort == 0 ? translate("Unordered") : tooltipSort == 1 ? translate("Descending") : translate("Ascending") |1216|1216| }); |1217|1217| | | [NORMAL] ESLintBear (indent): | | Expected indentation of 3 tabs but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/session.js |1212|1212| let orderHotkeyTooltip = Object.keys(viewablePlayerStates).length <= 1 ? "" : |1213|1213| "\n" + sprintf(translate("%(order)s: %(hotkey)s to change order."), { |1214|1214| "hotkey": setStringTags("\\[Click]", g_HotkeyTags), |1215| |- "order": tooltipSort == 0 ? translate("Unordered") : tooltipSort == 1 ? translate("Descending") : translate("Ascending") | |1215|+ "order": tooltipSort == 0 ? translate("Unordered") : tooltipSort == 1 ? translate("Descending") : translate("Ascending") |1216|1216| }); |1217|1217| |1218|1218| let resCodes = g_ResourceData.GetCodes(); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/session.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/session.js |1213|1213| "\n" + sprintf(translate("%(order)s: %(hotkey)s to change order."), { |1214|1214| "hotkey": setStringTags("\\[Click]", g_HotkeyTags), |1215|1215| "order": tooltipSort == 0 ? translate("Unordered") : tooltipSort == 1 ? translate("Descending") : translate("Ascending") |1216| |- }); | |1216|+ }); |1217|1217| |1218|1218| let resCodes = g_ResourceData.GetCodes(); |1219|1219| for (let r = 0; r < resCodes.length; ++r) binaries/data/mods/public/gui/session/session.js |1072| » let·getPanelEntNameTooltip·=·panelEntState·=>·"[font=\"sans-bold-16\"]"·+·template.name.specific·+·"[/font]"; | | [NORMAL] ESLintBear (no-shadow): | | 'panelEntState' is already declared in the upper scope. binaries/data/mods/public/gui/session/session.js |1147| » » button.onpress·=·(function(i)·{·return·function()·{·performGroup((Engine.HotkeyIsPressed("selection.add")·?·"add"·:·"select"),·i);·};·})(i); | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/session.js |1148| » » button.ondoublepress·=·(function(i)·{·return·function()·{·performGroup("snap",·i);·};·})(i); | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/session.js |1149| » » button.onpressright·=·(function(i)·{·return·function()·{·performGroup("breakUp",·i);·};·})(i); | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. | | [NORMAL] ESLintBear (space-before-function-paren): | | Unexpected space before function parentheses. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/messages.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/messages.js | 467| 467| g_Selection.reset(); | 468| 468| g_Selection.addList(selection, false, cmd.type == "gather"); | 469| 469| }, | 470| |- "play-tracks": function (notification, player) | | 470|+ "play-tracks": function(notification, player) | 471| 471| { | 472| 472| if (notification.lock) | 473| 473| { | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/messages.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/messages.js | 574| 574| let notificationText = | 575| 575| notification.instructions.reduce((instructions, item) => | 576| 576| instructions + (typeof item == "string" ? translate(item) : colorizeHotkey(translate(item.text), item.hotkey)), | 577| |- ""); | | 577|+ ""); | 578| 578| | 579| 579| Engine.GetGUIObjectByName("tutorialText").caption = g_TutorialMessages.concat(setStringTags(notificationText, g_TutorialNewMessageTags)).join("\n"); | 580| 580| g_TutorialMessages.push(notificationText); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 192| 192| "tooltip": translate("Exits the game."), | 193| 193| "onPress": () => { | 194| 194| messageBox( | 195| |- 400, 200, | | 195|+ 400, 200, | 196| 196| translate("Are you sure you want to quit 0 A.D.?"), | 197| 197| translate("Confirmation"), | 198| 198| [translate("No"), translate("Yes")], | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 193| 193| "onPress": () => { | 194| 194| messageBox( | 195| 195| 400, 200, | 196| |- translate("Are you sure you want to quit 0 A.D.?"), | | 196|+ translate("Are you sure you want to quit 0 A.D.?"), | 197| 197| translate("Confirmation"), | 198| 198| [translate("No"), translate("Yes")], | 199| 199| [null, Engine.Exit]); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 194| 194| messageBox( | 195| 195| 400, 200, | 196| 196| translate("Are you sure you want to quit 0 A.D.?"), | 197| |- translate("Confirmation"), | | 197|+ translate("Confirmation"), | 198| 198| [translate("No"), translate("Yes")], | 199| 199| [null, Engine.Exit]); | 200| 200| } | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 195| 195| 400, 200, | 196| 196| translate("Are you sure you want to quit 0 A.D.?"), | 197| 197| translate("Confirmation"), | 198| |- [translate("No"), translate("Yes")], | | 198|+ [translate("No"), translate("Yes")], | 199| 199| [null, Engine.Exit]); | 200| 200| } | 201| 201| } | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 196| 196| translate("Are you sure you want to quit 0 A.D.?"), | 197| 197| translate("Confirmation"), | 198| 198| [translate("No"), translate("Yes")], | 199| |- [null, Engine.Exit]); | | 199|+ [null, Engine.Exit]); | 200| 200| } | 201| 201| } | 202| 202| ]; | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 62| 62| var g_RomanNumbers = [undefined, "I", "II", "III", "IV", "V", "VI", "VII", "VIII"]; | 63| 63| | 64| 64| var g_PlayerTeamList = prepareForDropdown([{ | 65| |- "label": translateWithContext("team", "None"), | | 65|+ "label": translateWithContext("team", "None"), | 66| 66| "id": -1 | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 63| 63| | 64| 64| var g_PlayerTeamList = prepareForDropdown([{ | 65| 65| "label": translateWithContext("team", "None"), | 66| |- "id": -1 | | 66|+ "id": -1 | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 64| 64| var g_PlayerTeamList = prepareForDropdown([{ | 65| 65| "label": translateWithContext("team", "None"), | 66| 66| "id": -1 | 67| |- }].concat( | | 67|+}].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| 70| "id": i | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 65| 65| "label": translateWithContext("team", "None"), | 66| 66| "id": -1 | 67| 67| }].concat( | 68| |- Array(g_MaxTeams).fill(0).map((v, i) => ({ | | 68|+ Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| 70| "id": i | 71| 71| })) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 66| 66| "id": -1 | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| |- "label": i + 1, | | 69|+ "label": i + 1, | 70| 70| "id": i | 71| 71| })) | 72| 72| ) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| |- "id": i | | 70|+ "id": i | 71| 71| })) | 72| 72| ) | 73| 73| ); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| 70| "id": i | 71| |- })) | | 71|+ })) | 72| 72| ) | 73| 73| ); | 74| 74| | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 69| 69| "label": i + 1, | 70| 70| "id": i | 71| 71| })) | 72| |- ) | | 72|+) | 73| 73| ); | 74| 74| | 75| 75| /** | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 78| 78| var g_RelicCountList = Object.keys(g_CivData).map((civ, i) => i + 1); | 79| 79| | 80| 80| var g_PlayerCivList = g_CivData && prepareForDropdown([{ | 81| |- "name": translateWithContext("civilization", "Random"), | | 81|+ "name": translateWithContext("civilization", "Random"), | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 79| 79| | 80| 80| var g_PlayerCivList = g_CivData && prepareForDropdown([{ | 81| 81| "name": translateWithContext("civilization", "Random"), | 82| |- "tooltip": translate("Picks one civilization at random when the game starts."), | | 82|+ "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | 85| 85| }].concat( | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 80| 80| var g_PlayerCivList = g_CivData && prepareForDropdown([{ | 81| 81| "name": translateWithContext("civilization", "Random"), | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| |- "color": g_ColorRandom, | | 83|+ "color": g_ColorRandom, | 84| 84| "code": "random" | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 81| 81| "name": translateWithContext("civilization", "Random"), | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| |- "code": "random" | | 84|+ "code": "random" | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | 85| |- }].concat( | | 85|+}].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | 85| 85| }].concat( | 86| |- Object.keys(g_CivData).filter( | | 86|+ Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 84| 84| "code": "random" | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| |- civ => g_CivData[civ].SelectableInGameSetup | | 87|+ civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| |- ).map(civ => ({ | | 88|+ ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| |- "name": g_CivData[civ].Name, | | 89|+ "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| |- "tooltip": g_CivData[civ].History, | | 90|+ "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| |- "color": g_ColorRegular, | | 91|+ "color": g_ColorRegular, | 92| 92| "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | 94| 94| ) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| |- "code": civ | | 92|+ "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | 94| 94| ) | 95| 95| ); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | 93| |- })).sort(sortNameIgnoreCase) | | 93|+ })).sort(sortNameIgnoreCase) | 94| 94| ) | 95| 95| ); | 96| 96| | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | 94| |- ) | | 94|+) | 95| 95| ); | 96| 96| | 97| 97| /** | | [NORMAL] ESLintBear (no-trailing-spaces): | | Trailing spaces not allowed. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js |1328|1328| offset = -Math.min(slideSpeed * dt, maxOffset); |1329|1329| } |1330|1330| |1331| |- updateSettingsPanelPosition(offset); | |1331|+ updateSettingsPanelPosition(offset); |1332|1332| } |1333|1333| |1334|1334| /** Executing section cli...
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/780/display/redirect
binaries/data/mods/public/gui/pregame/MainMenuItems.js | ||
---|---|---|
50 | Good question! Do you happen to know if and where 0 A.D. manual of style on title case is specified? | |
80 | What does this line do? Is it also a user-facing text string, or something else? | |
binaries/data/mods/public/gui/replaymenu/replay_filters.js | ||
143 | What does this line do? Is it also a user-facing text string, or something else? |
Single-player is proper English, single player and singleplayer are less correct.
I don’t think that’s true. I believe “single player” is the right wording in general. An hyphen may be used when “single player” is used as an adjective and not using an hyphen may allow for wrong interpretations. For example, «single-player campaign» may use an hyphen to make sure it is not understood as “one campaign of players”. For when used standalone, for example as a button label, I don’t think there’s a reason to use an hyphen.
The hyphen is there to avoid ambiguity. I don't think in any of the lines changed here “single player” or “singeplayer” would be better than “single-player”.
Also, for what's worth:
https://www.lexico.com/en/definition/single-player
https://www.lexico.com/en/definition/multiplayer
https://en.wikipedia.org/wiki/Hyphen#Varied_meanings
I believe this confirms what I’m saying. ‘Single-player’ only works as an adjective.
I disagree. Wherever ‘single-player’ is not followed by a noun, it feels a bit off to me. ‘Single-Player Game’ and ‘Single Player’ look good to me as button captions, for example, but ‘Single-Player’ does not.
Yes, single-player is an adjective, as is multiplayer; I don't think I claimed otherwise.
Please have a careful look at the affected lines and point out which ought to be changed.
I believe in none these cases the meaning is as in “He is a single player.”
Yeah, you are right. It’s just that I’m not used to seing “Single-player” on its own in a button, but it’s correct, the noun (game mode) is just omitted. I’ve left a couple of suggestions, but I’m OK with merging as is. I’ll probably merge it next weekend.
binaries/data/mods/public/gui/options/options.json | ||
---|---|---|
8–9 | Player name | |
binaries/data/mods/public/gui/replaymenu/replay_filters.js | ||
142 | “Single-player and multiplayer?” |
binaries/data/mods/public/gui/pregame/MainMenuItems.js | ||
---|---|---|
80 | This is the standard filter applied in the replay menu. (See replay_filters.js L143.) It is not user-facing. | |
binaries/data/mods/public/gui/replaymenu/replay_filters.js | ||
143 | This is the data belonging to the entries specified above. The filter "" is (internally) correlated to the (user facing) option translate("Single- and multiplayer") in the list above. So not use-facing. |
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/588/display/redirect
binaries/data/mods/public/gui/options/options.json | ||
---|---|---|
8–9 | There are probably more occurrences of “playername”. Perhaps something for a future patch. |
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues: Executing section Source... Executing section JS... | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 193| 193| "tooltip": translate("Exits the game."), | 194| 194| "onPress": () => { | 195| 195| messageBox( | 196| |- 400, 200, | | 196|+ 400, 200, | 197| 197| translate("Are you sure you want to quit 0 A.D.?"), | 198| 198| translate("Confirmation"), | 199| 199| [translate("No"), translate("Yes")], | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 194| 194| "onPress": () => { | 195| 195| messageBox( | 196| 196| 400, 200, | 197| |- translate("Are you sure you want to quit 0 A.D.?"), | | 197|+ translate("Are you sure you want to quit 0 A.D.?"), | 198| 198| translate("Confirmation"), | 199| 199| [translate("No"), translate("Yes")], | 200| 200| [null, Engine.Exit]); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 195| 195| messageBox( | 196| 196| 400, 200, | 197| 197| translate("Are you sure you want to quit 0 A.D.?"), | 198| |- translate("Confirmation"), | | 198|+ translate("Confirmation"), | 199| 199| [translate("No"), translate("Yes")], | 200| 200| [null, Engine.Exit]); | 201| 201| } | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 196| 196| 400, 200, | 197| 197| translate("Are you sure you want to quit 0 A.D.?"), | 198| 198| translate("Confirmation"), | 199| |- [translate("No"), translate("Yes")], | | 199|+ [translate("No"), translate("Yes")], | 200| 200| [null, Engine.Exit]); | 201| 201| } | 202| 202| } | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 197| 197| translate("Are you sure you want to quit 0 A.D.?"), | 198| 198| translate("Confirmation"), | 199| 199| [translate("No"), translate("Yes")], | 200| |- [null, Engine.Exit]); | | 200|+ [null, Engine.Exit]); | 201| 201| } | 202| 202| } | 203| 203| ]; | | [NORMAL] ESLintBear (key-spacing): | | Extra space before value for key 'onPress'. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/message_box/QuitConfirmationMenu.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/message_box/QuitConfirmationMenu.js | 50| 50| }, | 51| 51| { | 52| 52| "caption": translate("Yes"), | 53| |- "onPress": () => { | | 53|+ "onPress": () => { | 54| 54| (new ReturnQuestion()).display(); | 55| 55| } | 56| 56| } binaries/data/mods/public/gui/session/message_box/QuitConfirmationMenu.js | 54| » » » (new·ReturnQuestion()).display(); | | [MAJOR] ESLintBear (no-use-before-define): | | 'ReturnQuestion' was used before it was defined. binaries/data/mods/public/gui/session/message_box/QuitConfirmationMenu.js | 59| class·ReturnQuestion·extends·SessionMessageBox | | [NORMAL] JSHintBear: | | 'ReturnQuestion' was used before it was defined. binaries/data/mods/public/gui/replaymenu/replay_filters.js | 188| » » switch·(sortKey) | | [NORMAL] ESLintBear (default-case): | | Expected a default case. | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 62| 62| var g_RomanNumbers = [undefined, "I", "II", "III", "IV", "V", "VI", "VII", "VIII"]; | 63| 63| | 64| 64| var g_PlayerTeamList = prepareForDropdown([{ | 65| |- "label": translateWithContext("team", "None"), | | 65|+ "label": translateWithContext("team", "None"), | 66| 66| "id": -1 | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 63| 63| | 64| 64| var g_PlayerTeamList = prepareForDropdown([{ | 65| 65| "label": translateWithContext("team", "None"), | 66| |- "id": -1 | | 66|+ "id": -1 | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 64| 64| var g_PlayerTeamList = prepareForDropdown([{ | 65| 65| "label": translateWithContext("team", "None"), | 66| 66| "id": -1 | 67| |- }].concat( | | 67|+}].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| 70| "id": i | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 65| 65| "label": translateWithContext("team", "None"), | 66| 66| "id": -1 | 67| 67| }].concat( | 68| |- Array(g_MaxTeams).fill(0).map((v, i) => ({ | | 68|+ Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| 70| "id": i | 71| 71| })) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 66| 66| "id": -1 | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| |- "label": i + 1, | | 69|+ "label": i + 1, | 70| 70| "id": i | 71| 71| })) | 72| 72| ) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| |- "id": i | | 70|+ "id": i | 71| 71| })) | 72| 72| ) | 73| 73| ); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| 70| "id": i | 71| |- })) | | 71|+ })) | 72| 72| ) | 73| 73| ); | 74| 74| | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 69| 69| "label": i + 1, | 70| 70| "id": i | 71| 71| })) | 72| |- ) | | 72|+) | 73| 73| ); | 74| 74| | 75| 75| /** | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 78| 78| var g_RelicCountList = Object.keys(g_CivData).map((civ, i) => i + 1); | 79| 79| | 80| 80| var g_PlayerCivList = g_CivData && prepareForDropdown([{ | 81| |- "name": translateWithContext("civilization", "Random"), | | 81|+ "name": translateWithContext("civilization", "Random"), | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 79| 79| | 80| 80| var g_PlayerCivList = g_CivData && prepareForDropdown([{ | 81| 81| "name": translateWithContext("civilization", "Random"), | 82| |- "tooltip": translate("Picks one civilization at random when the game starts."), | | 82|+ "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | 85| 85| }].concat( | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 80| 80| var g_PlayerCivList = g_CivData && prepareForDropdown([{ | 81| 81| "name": translateWithContext("civilization", "Random"), | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| |- "color": g_ColorRandom, | | 83|+ "color": g_ColorRandom, | 84| 84| "code": "random" | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 81| 81| "name": translateWithContext("civilization", "Random"), | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| |- "code": "random" | | 84|+ "code": "random" | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | 85| |- }].concat( | | 85|+}].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | 85| 85| }].concat( | 86| |- Object.keys(g_CivData).filter( | | 86|+ Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 84| 84| "code": "random" | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| |- civ => g_CivData[civ].SelectableInGameSetup | | 87|+ civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| |- ).map(civ => ({ | | 88|+ ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| |- "name": g_CivData[civ].Name, | | 89|+ "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| |- "tooltip": g_CivData[civ].History, | | 90|+ "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| |- "color": g_ColorRegular, | | 91|+ "color": g_ColorRegular, | 92| 92| "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | 94| 94| ) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| |- "code": civ | | 92|+ "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | 94| 94| ) | 95| 95| ); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | 93| |- })).sort(sortNameIgnoreCase) | | 93|+ })).sort(sortNameIgnoreCase) | 94| 94| ) | 95| 95| ); | 96| 96| | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | 94| |- ) | | 94|+) | 95| 95| ); | 96| 96| | 97| 97| /** | | [NORMAL] ESLintBear (no-trailing-spaces): | | Trailing spaces not allowed. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js |1328|1328| offset = -Math.min(slideSpeed * dt, maxOffset); |1329|1329| } |1330|1330| |1331| |- updateSettingsPanelPosition(offset); | |1331|+ updateSettingsPanelPosition(offset); |1332|1332| } |1333|1333| |1334|1334| /** binaries/data/mods/public/gui/session/session.js | 686| » » button.onpress·=·(function(i)·{·return·function()·{·performGroup((Engine.HotkeyIsPressed("selection.add")·?·"add"·:·"select"),·i);·};·})(i); | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/session.js | 687| » » button.ondoublepress·=·(function(i)·{·return·function()·{·performGroup("snap",·i);·};·})(i); | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/session.js | 688| » » button.onpressright·=·(function(i)·{·return·function()·{·performGroup("breakUp",·i);·};·})(i); | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. Executing section cli...
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1104/display/redirect
binaries/data/mods/public/gui/options/options.json | ||
---|---|---|
8–9 | It seems that every option is Title Case, do you plan on revisiting that? It would look out of place to not use it here (and below). | |
binaries/data/mods/public/gui/replaymenu/replay_menu.xml | ||
113 ↗ | (On Diff #10340) | I guess this _could_ be changed, but I think hypens are not used in GUI-object names? @elexis, your opinion/knowledge? |
binaries/data/mods/public/gui/options/options.json | ||
---|---|---|
8–9 | If it's a button caption, then yes, it ought to be in title case. |
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/592/display/redirect
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues: Executing section Source... Executing section JS... | | [NORMAL] ESLintBear (key-spacing): | | Extra space before value for key 'onPress'. |----| | /zpool0/trunk/binaries/data/mods/public/gui/session/message_box/QuitConfirmationMenu.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/session/message_box/QuitConfirmationMenu.js | 50| 50| }, | 51| 51| { | 52| 52| "caption": translate("Yes"), | 53| |- "onPress": () => { | | 53|+ "onPress": () => { | 54| 54| (new ReturnQuestion()).display(); | 55| 55| } | 56| 56| } binaries/data/mods/public/gui/session/message_box/QuitConfirmationMenu.js | 54| » » » (new·ReturnQuestion()).display(); | | [MAJOR] ESLintBear (no-use-before-define): | | 'ReturnQuestion' was used before it was defined. binaries/data/mods/public/gui/session/message_box/QuitConfirmationMenu.js | 59| class·ReturnQuestion·extends·SessionMessageBox | | [NORMAL] JSHintBear: | | 'ReturnQuestion' was used before it was defined. binaries/data/mods/public/gui/replaymenu/replay_filters.js | 188| » » switch·(sortKey) | | [NORMAL] ESLintBear (default-case): | | Expected a default case. | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 62| 62| var g_RomanNumbers = [undefined, "I", "II", "III", "IV", "V", "VI", "VII", "VIII"]; | 63| 63| | 64| 64| var g_PlayerTeamList = prepareForDropdown([{ | 65| |- "label": translateWithContext("team", "None"), | | 65|+ "label": translateWithContext("team", "None"), | 66| 66| "id": -1 | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 63| 63| | 64| 64| var g_PlayerTeamList = prepareForDropdown([{ | 65| 65| "label": translateWithContext("team", "None"), | 66| |- "id": -1 | | 66|+ "id": -1 | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 64| 64| var g_PlayerTeamList = prepareForDropdown([{ | 65| 65| "label": translateWithContext("team", "None"), | 66| 66| "id": -1 | 67| |- }].concat( | | 67|+}].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| 70| "id": i | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 65| 65| "label": translateWithContext("team", "None"), | 66| 66| "id": -1 | 67| 67| }].concat( | 68| |- Array(g_MaxTeams).fill(0).map((v, i) => ({ | | 68|+ Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| 70| "id": i | 71| 71| })) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 66| 66| "id": -1 | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| |- "label": i + 1, | | 69|+ "label": i + 1, | 70| 70| "id": i | 71| 71| })) | 72| 72| ) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 67| 67| }].concat( | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| |- "id": i | | 70|+ "id": i | 71| 71| })) | 72| 72| ) | 73| 73| ); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 68| 68| Array(g_MaxTeams).fill(0).map((v, i) => ({ | 69| 69| "label": i + 1, | 70| 70| "id": i | 71| |- })) | | 71|+ })) | 72| 72| ) | 73| 73| ); | 74| 74| | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 69| 69| "label": i + 1, | 70| 70| "id": i | 71| 71| })) | 72| |- ) | | 72|+) | 73| 73| ); | 74| 74| | 75| 75| /** | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 78| 78| var g_RelicCountList = Object.keys(g_CivData).map((civ, i) => i + 1); | 79| 79| | 80| 80| var g_PlayerCivList = g_CivData && prepareForDropdown([{ | 81| |- "name": translateWithContext("civilization", "Random"), | | 81|+ "name": translateWithContext("civilization", "Random"), | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 79| 79| | 80| 80| var g_PlayerCivList = g_CivData && prepareForDropdown([{ | 81| 81| "name": translateWithContext("civilization", "Random"), | 82| |- "tooltip": translate("Picks one civilization at random when the game starts."), | | 82|+ "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | 85| 85| }].concat( | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 80| 80| var g_PlayerCivList = g_CivData && prepareForDropdown([{ | 81| 81| "name": translateWithContext("civilization", "Random"), | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| |- "color": g_ColorRandom, | | 83|+ "color": g_ColorRandom, | 84| 84| "code": "random" | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 81| 81| "name": translateWithContext("civilization", "Random"), | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| |- "code": "random" | | 84|+ "code": "random" | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 82| 82| "tooltip": translate("Picks one civilization at random when the game starts."), | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | 85| |- }].concat( | | 85|+}].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 83| 83| "color": g_ColorRandom, | 84| 84| "code": "random" | 85| 85| }].concat( | 86| |- Object.keys(g_CivData).filter( | | 86|+ Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 84| 84| "code": "random" | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| |- civ => g_CivData[civ].SelectableInGameSetup | | 87|+ civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 85| 85| }].concat( | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| |- ).map(civ => ({ | | 88|+ ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 86| 86| Object.keys(g_CivData).filter( | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| |- "name": g_CivData[civ].Name, | | 89|+ "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 87| 87| civ => g_CivData[civ].SelectableInGameSetup | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| |- "tooltip": g_CivData[civ].History, | | 90|+ "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 88| 88| ).map(civ => ({ | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| |- "color": g_ColorRegular, | | 91|+ "color": g_ColorRegular, | 92| 92| "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | 94| 94| ) | | [NORMAL] ESLintBear (indent): | | Expected indentation of 2 tabs but found 3. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 89| 89| "name": g_CivData[civ].Name, | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| |- "code": civ | | 92|+ "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | 94| 94| ) | 95| 95| ); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 1 tab but found 2. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 90| 90| "tooltip": g_CivData[civ].History, | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | 93| |- })).sort(sortNameIgnoreCase) | | 93|+ })).sort(sortNameIgnoreCase) | 94| 94| ) | 95| 95| ); | 96| 96| | | [NORMAL] ESLintBear (indent): | | Expected indentation of 0 tabs but found 1. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | 91| 91| "color": g_ColorRegular, | 92| 92| "code": civ | 93| 93| })).sort(sortNameIgnoreCase) | 94| |- ) | | 94|+) | 95| 95| ); | 96| 96| | 97| 97| /** | | [NORMAL] ESLintBear (no-trailing-spaces): | | Trailing spaces not allowed. |----| | /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/gamesetup/gamesetup.js |1328|1328| offset = -Math.min(slideSpeed * dt, maxOffset); |1329|1329| } |1330|1330| |1331| |- updateSettingsPanelPosition(offset); | |1331|+ updateSettingsPanelPosition(offset); |1332|1332| } |1333|1333| |1334|1334| /** | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 193| 193| "tooltip": translate("Exits the game."), | 194| 194| "onPress": () => { | 195| 195| messageBox( | 196| |- 400, 200, | | 196|+ 400, 200, | 197| 197| translate("Are you sure you want to quit 0 A.D.?"), | 198| 198| translate("Confirmation"), | 199| 199| [translate("No"), translate("Yes")], | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 194| 194| "onPress": () => { | 195| 195| messageBox( | 196| 196| 400, 200, | 197| |- translate("Are you sure you want to quit 0 A.D.?"), | | 197|+ translate("Are you sure you want to quit 0 A.D.?"), | 198| 198| translate("Confirmation"), | 199| 199| [translate("No"), translate("Yes")], | 200| 200| [null, Engine.Exit]); | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 195| 195| messageBox( | 196| 196| 400, 200, | 197| 197| translate("Are you sure you want to quit 0 A.D.?"), | 198| |- translate("Confirmation"), | | 198|+ translate("Confirmation"), | 199| 199| [translate("No"), translate("Yes")], | 200| 200| [null, Engine.Exit]); | 201| 201| } | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 196| 196| 400, 200, | 197| 197| translate("Are you sure you want to quit 0 A.D.?"), | 198| 198| translate("Confirmation"), | 199| |- [translate("No"), translate("Yes")], | | 199|+ [translate("No"), translate("Yes")], | 200| 200| [null, Engine.Exit]); | 201| 201| } | 202| 202| } | | [NORMAL] ESLintBear (indent): | | Expected indentation of 4 tabs but found 5. |----| | /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | |++++| /zpool0/trunk/binaries/data/mods/public/gui/pregame/MainMenuItems.js | 197| 197| translate("Are you sure you want to quit 0 A.D.?"), | 198| 198| translate("Confirmation"), | 199| 199| [translate("No"), translate("Yes")], | 200| |- [null, Engine.Exit]); | | 200|+ [null, Engine.Exit]); | 201| 201| } | 202| 202| } | 203| 203| ]; binaries/data/mods/public/gui/session/session.js | 686| » » button.onpress·=·(function(i)·{·return·function()·{·performGroup((Engine.HotkeyIsPressed("selection.add")·?·"add"·:·"select"),·i);·};·})(i); | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/session.js | 687| » » button.ondoublepress·=·(function(i)·{·return·function()·{·performGroup("snap",·i);·};·})(i); | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. binaries/data/mods/public/gui/session/session.js | 688| » » button.onpressright·=·(function(i)·{·return·function()·{·performGroup("breakUp",·i);·};·})(i); | | [NORMAL] ESLintBear (no-shadow): | | 'i' is already declared in the upper scope. Executing section cli...
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1108/display/redirect