Page MenuHomePhabricator

Get repo code somewhat up to date with current app
AbandonedPublic

Authored by WyldBot on Apr 20 2019, 12:41 PM.

Details

Summary

Original iOS repo

Fixes

Test Plan

Nothing, just review/accept

Diff Detail

Repository
rIOS iOS Client
Branch
master
Lint
Lint Errors
SeverityLocationCodeMessage
ErrorWildFyre.xcodeproj\project.pbxproj:3TXT2Tab Literal
ErrorWildFyre.xcodeproj\xcuserdata\user909400.xcuserdatad\xcschemes\xcschememanagement.plist:5TXT2Tab Literal
WarningWildFyre.xcodeproj\project.pbxproj:10TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:11TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:12TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:13TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:14TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:15TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:16TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:17TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:18TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:39TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:40TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:41TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:42TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:43TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:44TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:45TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:46TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:47TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:48TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:49TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:50TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:51TXT3Line Too Long
WarningWildFyre.xcodeproj\project.pbxproj:170TXT3Line Too Long
Unit
No Test Coverage
Build Status
Buildable 633
Build 633: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Apr 22 2019, 6:17 PM

There is a big warning above the box and still I nearly missed it....

What is with lint?

This revision now requires changes to proceed.Apr 22 2019, 6:18 PM
  • New iosclient with code
  • New iosclient with code
  • New iosclient with code
  • Removed private files
  • Fixed errors
  1. Updating D244: Get repo code somewhat up to date with current app #
  2. Enter a brief description of the changes included in this update.
  3. The first line is used as subject, next lines as comment. #

Current ios client
The one that's on the app store

Info-Screen abandoned this revision.
Info-Screen reclaimed this revision.
Info-Screen edited reviewers, added: SweatyMushroomz; removed: Info-Screen.

Fu*k wrong revision too.....

Lint lines are too long. Check my inline comment in AlertChildPageViewController.xib:2

Pods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib
2 ↗(On Diff #876)

Split the line at a convenient location. It's not hard, just press enter instead of space.
Repeat for all the other warnings

WyldBot requested changes to this revision.Apr 24 2019, 1:19 PM

Add a .gitignore to hide .DS_Store files and all files in the Pods folder

This revision now requires changes to proceed.Apr 24 2019, 1:19 PM
In D244#6038, @WyldBot wrote:

Add a .gitignore to hide .DS_Store files and all files in the Pods folder

Also Podfile.lock

WyldBot edited reviewers, added: SweatyMushroomz; removed: WyldBot.
This revision now requires review to proceed.Apr 25 2019, 4:23 AM
  • Git ignore
  • fixed untracked files