Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Declare some more classes, functions and variables as open #666

Merged
merged 1 commit into from
Sep 16, 2016

Conversation

mats-claassen
Copy link
Member

fixes #646
fixes #643

@xmartlabs/eureka

@@ -137,7 +137,7 @@ extension BaseRow {
/**
Evaluates if the row should be hidden or not and updates the form accordingly
*/
public final func evaluateHidden() {
public func evaluateHidden() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keep final

@@ -156,7 +156,7 @@ extension BaseRow {
/**
Evaluates if the row should be disabled or not and updates it accordingly
*/
public final func evaluateDisabled() {
public func evaluateDisabled() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keep final please

@@ -549,7 +549,7 @@ open class FormViewController : UIViewController, FormViewControllerProtocol {
/**
Called when a cell becomes first responder
*/
public final func beginEditing<T:Equatable>(_ cell: Cell<T>) {
public func beginEditing<T:Equatable>(_ cell: Cell<T>) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here too

@@ -567,7 +567,7 @@ open class FormViewController : UIViewController, FormViewControllerProtocol {
/**
Called when a cell resigns first responder
*/
public final func endEditing<T:Equatable>(_ cell: Cell<T>) {
public func endEditing<T:Equatable>(_ cell: Cell<T>) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

too

@mtnbarreto mtnbarreto merged commit c2b0b03 into Swift3 Sep 16, 2016
@mtnbarreto mtnbarreto deleted the openClasses branch September 16, 2016 21:45
@mtnbarreto mtnbarreto added this to the 2.0.0 milestone Sep 19, 2016
@RigoOnRails
Copy link

This is an evil pull request lmao

@mtnbarreto
Copy link
Member

hahaha. it was merged by me 😱

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants