-
Notifications
You must be signed in to change notification settings - Fork 48
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
[ci] Added Github CI Build #83
Conversation
a4682ee
to
ea1f4a2
Compare
So reordering some classes fix the error. 🤔 Thanks for the help @purhan 😄 |
@devkapilbansal The build previously failed because of this change: openwisp/openwisp-users#230. |
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.
Thanks @devkapilbansal, the travis config file has to be removed though, can you please do that?
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.
A possible follow up to improve testing: #85
073f0b1
to
11e1828
Compare
Closes #85
11e1828
to
9af3d19
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.
Thanks, Looks good! 😄
(I have not tested it)
@@ -1,5 +1,5 @@ | |||
# TODO: change this when next version of openwisp_users is released | |||
openwisp-users @ https://github.com/purhan/openwisp-users/tarball/master | |||
openwisp-users @ https://github.com/openwisp/openwisp-users/tarball/master |
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.
@purhan was there a reason for using your fork or was this a mistake? 😄
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.
I believe it was a leftover
@@ -6,6 +6,10 @@ echo '' | |||
echo 'Check JavaScript Linting' | |||
jslint ./openwisp_ipam/static/openwisp-ipam/js/*.js | |||
|
|||
echo '' | |||
echo 'Check CSS Linting' | |||
stylelint ./openwisp_ipam/static/openwisp-ipam/css/*.css |
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.
Perhaps we should move something like this in openwisp-utils and run it on all modules! 😄
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.
Great work, thanks @devkapilbansal
No description provided.