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

Code factor style issues fix #2135

Merged
merged 9 commits into from
Aug 24, 2017
Merged

Code factor style issues fix #2135

merged 9 commits into from
Aug 24, 2017

Conversation

Prandtl
Copy link
Contributor

@Prandtl Prandtl commented Aug 23, 2017

✨ What kind of change does this PR introduce? (Bug fix, feature, docs update...)

Code factor styling issues fix for #1922

⤵️ What is the current behavior?

🆕 What is the new behavior (if this is a feature change)?

💥 Does this PR introduce a breaking change?

No

🐛 Recommendations for testing

📝 Links to relevant issues/docs

🤔 Checklist before submitting

  • All projects build
  • Follows style guide lines (code style guide)
  • Relevant documentation was updated (docs style guide)
  • Nuspec files were updated (when applicable)
  • Rebased onto current develop

@Prandtl
Copy link
Contributor Author

Prandtl commented Aug 23, 2017

Don't really know what to do about code duplication issues: the code duplication is mostly in Droid controls and I don't see a nice way and a reason to remove this duplication. If there are no suggestions this can be merged.

@martijn00
Copy link
Contributor

Can you make sure to use spaces instead of tabs?

@Prandtl
Copy link
Contributor Author

Prandtl commented Aug 23, 2017

Changed tabs to spaces.

@martijn00
Copy link
Contributor

Some are still tabs. After that is fixed i think this is good to merge.

@martijn00 martijn00 added this to the 5.2.0 milestone Aug 23, 2017
@Prandtl
Copy link
Contributor Author

Prandtl commented Aug 23, 2017

OK, done. By the way, how do you check PR if it is spaces only?

@martijn00 martijn00 merged commit a321245 into MvvmCross:develop Aug 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants