-
Notifications
You must be signed in to change notification settings - Fork 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
[qa] Remove bad code practices #129 #133
Conversation
140d9fc
to
d0d1c53
Compare
.editorconfig
Outdated
@@ -9,3 +9,4 @@ end_of_line = lf | |||
charset = utf-8 | |||
trim_trailing_whitespace = true | |||
insert_final_newline = true | |||
quote_type = single |
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.
why this? We don't have this in https://github.com/openwisp/openwisp-wifi-login-pages/blob/master/.editorconfig .
We can keep double quotes I believe.
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 thought it would be nice to have single quotes as we are using single quotes on other modules. I will change it back.
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 also rebase on the latest gsoc22 branch.
d0d1c53
to
abc6dfb
Compare
abc6dfb
to
34afec4
Compare
closes #129