-
Notifications
You must be signed in to change notification settings - Fork 23
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
First PR! Started adding structure for the reddit object implementation #1
Conversation
draw/reddit.dart
Outdated
@@ -0,0 +1,9 @@ | |||
// Provides the reddit class | |||
|
|||
class reddit{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Take a look at this page here for documentation styling: https://www.dartlang.org/guides/language/effective-dart/documentation. If we do it right, dartdoc automatically generates documentation pages when we upload the library to pub!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just minor nitpicking.
draw/reddit.dart
Outdated
@@ -0,0 +1,9 @@ | |||
// Provides the reddit class | |||
|
|||
class reddit{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please be sure to run dartfmt and analyzer on your code before submitting.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done, following was the output for dart analyser:
$ Analyzing [reddit.dart]...
$ No issues found
draw/reddit.dart
Outdated
class reddit{ | ||
|
||
//Reddit Constructor | ||
reddit(){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typical style for Dart looks like this:
if (foo) {
// Note the space between the closing paren and opening brace.
}
…lar comments with adherence to dart docs.
draw/reddit.dart
Outdated
} | ||
class reddit { | ||
reddit() { | ||
///This is currently a placeholder for the Reddit Constructor. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please always put a space before text in comments.
#Core Functionality
Yes
#Description
Added a draw folder to contain configuration and the reddit object implementation that will be exposed to users. Will also need some sort of folder for a replica of Model in praw. Recommending we keep configs in reddit.draw for now so that we can something up and running quickly. :)