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
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:2TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:22TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:25TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:27TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:28TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:31TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:33TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:34TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:35TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:40TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:41TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:42TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:43TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:44TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:45TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:46TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:49TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:52TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:57TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:58TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:59TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:60TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:65TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:66TXT3Line Too Long
WarningPods/AlertOnboarding/Pod/Assets/AlertChildPageViewController.xib:67TXT3Line Too Long
Unit
No Test Coverage
Build Status
Buildable 641
Build 641: 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
3

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