Skip to content

General code cleanups. #77

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

Merged
merged 5 commits into from
Feb 8, 2019
Merged

Conversation

chrbayer
Copy link
Contributor

@chrbayer chrbayer commented Feb 8, 2019

Some minor bugs fixed and some spellings corrected.

@chrbayer
Copy link
Contributor Author

chrbayer commented Feb 8, 2019

Please review before merging :-)

@@ -58,11 +58,11 @@ part 'src/utils/parse_utils.dart';

class Parse {
ParseCoreData data;
bool _hasBeenInitialised = false;
bool _hasBeenInitialized = false;
Copy link
Member

Choose a reason for hiding this comment

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

This is American English vs British - I'm happy either way to be fair.

Copy link
Member

@phillwiggins phillwiggins left a comment

Choose a reason for hiding this comment

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

Spelling isn't my strongest point, so thank you! (Although there's a few instances of American vs British English)

I appreciate your work.

@phillwiggins phillwiggins merged commit a578a01 into parse-community:master Feb 8, 2019
@chrbayer
Copy link
Contributor Author

chrbayer commented Feb 8, 2019

Thanks :-) I choose American English because my editor defaults to this variant of the language. And some functions like Parse().initialize were already in American English :-)
And I am not good at spelling, too, I need google and a good dictionary ;-)
Thanks for merging!

fischerscode pushed a commit to fischerscode/Parse-SDK-Flutter that referenced this pull request Sep 5, 2020
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.

2 participants