Closed Bug 1153081 Opened 9 years ago Closed 8 years ago

Fix nits from Bug 1152609 (Swift 1.2 changes)

Categories

(Firefox for iOS :: General, defect)

Product:
Firefox for iOS
Component:
General
Platform:
All
iOS 8
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
fxios-v5.0 --- fixed

People

(Reporter: rnewman, Assigned: aeriklawson, Mentored)

References

Details

(Whiteboard: [good first bug][lang=swift])

Attachments

(1 file, 1 obsolete file)

Pull request
48 bytes, text/x-github-pull-request
bnicholson
: review+
Details | Review 
https://github.com/mozilla/firefox-ios/pull/325

Wes and Brian had some nits.
I'd be willing to take these on.
Assignee: nobody → aeriklawson
Mentor: wjohnston2000
Status: NEW → ASSIGNED
Attached file Pull request (obsolete) — 

        Attachment #8741588 -
        Flags: review?(rnewman)

    
  

        Attachment #8741588 -
        Flags: review?(rnewman) → review?(bnicholson)

    
    

    
  
Comment on attachment 8741588 [details] [review]
Pull request

Thanks for the changes!

These changes look good for the most part, but note that you're working off of a very old HEAD (the parent commit to your changes is f5ab6aa, from April 2015). Please rebase your changes onto the latest master and force push to update this PR. Also, please squash all of these commits into one; they're all related to a single follow-up, so no need to split them.

After that, I think this should be good to go!

        Attachment #8741588 -
        Flags: review?(bnicholson) → feedback+

    
    

    
  
Geez, my brain thought it was still 2015 - everything makes much more sense now, haha. Okay, it was easier to just trash that branch and make a new one. Most of the changes were overtaken, but I did change a function signature that hadn't been fixed yet (some whitespace changes made their way in too).

    
    

    
  

      
      
      
      

        Attached file
          
        Pull request
        — 
      

    


  

        Attachment #8741909 -
        Flags: review?(bnicholson)

        Attachment #8741588 -
        Attachment is obsolete: true

    
    

    
  
https://github.com/mozilla/firefox-ios/commit/05946639b1eb2326a3f94e2a2edf31f085eb6d7c
Status: ASSIGNED → RESOLVED
Closed: 8 years ago

          status-fxios-v5.0:
          --- → fixed
Resolution: --- → FIXED

        Attachment #8741909 -
        Flags: review?(bnicholson) → review+

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: