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
[WorldMoodTracker] Initialized World Mood Tracker app #189
[WorldMoodTracker] Initialized World Mood Tracker app #189
Conversation
0aa63cd
to
3fbdd00
Compare
@daminisatya @djmgit @kavithaenair: Please review. |
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.
LGTM
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.
what do you say @daminisatya ?
@@ -1,7 +1,7 @@ | |||
[default] | |||
bears = LineLengthBear, SpaceConsistencyBear | |||
files = **.js, **.css, **.html, **.yml, **.json | |||
ignore = **.min.js, **.min.css, tweetheatmap/public/lib/**, webtweets/js/bootstrap/**, webtweets/css/bootstrap.css, js/typeahead.bundle.js, webtweets/js/bootstrap.js | |||
ignore = **.min.js, **.min.css, tweetheatmap/public/lib/**, webtweets/js/bootstrap/**, webtweets/css/bootstrap.css, js/typeahead.bundle.js, webtweets/js/bootstrap.js, **/node_modules/**, **/index_bundle.js, WorldMoodTracker/index.html |
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.
index.html and js files written by you should not be ignored by coala bears i guess.
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.
It is not like that. The two files ignored are generated by webpack
while the build process.
cd WorldMoodTracker
yarn install # If not already
yarn build
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.
Oh. Okay! 👍
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.
@@ -20,7 +20,7 @@ use_spaces = True | |||
[js] | |||
bears = PHPCodeSnifferBear | |||
files = **.js | |||
ignore = **.min.js, tweetheatmap/public/lib/**, /app/webtweets/js/bootstrap/** | |||
ignore = **.min.js, tweetheatmap/public/lib/**, /app/webtweets/js/bootstrap/**, **/node_modules/**, **/index_bundle.js |
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.
Here as well.
@Achint08 @hemantjadon: Please take a look. |
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.
Looks great 😊
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.
LGTM 👍
* Add Webpack, babel and related configurations * Ignore node_modules for sub apps * Add missing glyohicons font * Ignore dynamic files for coala
3fbdd00
to
d9d76d4
Compare
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.
@singhpratyush thank you for the code!
Great to see this getting started! Thank you! |
Short description
Fixes #188.
Changes:
Screenshots for the change:
I have:
Fixes #<number> commit message
For the reviewers
I have: