Linted files outside of `/app`

teknik-eksjo / chronos

A scheduling app that helps teachers submit workday outlines

MIT License

5 stars 5 forks source link

Closed Greenheart closed 8 years ago

Greenheart commented 8 years ago

Note: This doesn't fix the temporary linting-errors introduced in PR #106 since they will be fixed when that feature is completed.

Limpan commented 8 years ago

Can't the remaining errors be fixed too? It would be great to let Travis do something useful.

Greenheart commented 8 years ago

@Limpan All of them, or just the "normal" errors from manage.py lint?

Limpan commented 8 years ago

All of them. Or at least the ones in the tests directory. That way #130 won't be a pain.

Greenheart commented 8 years ago

@Limpan The rest of the linting-errors comes from /migrations/ and /docs/.

Should we ignore migrations-directory completely?
Is the conf-file in docs something we've created or is that too auto-generated --> If automatic, let's ignore /docs/ too?

Apart from those questions, this is ready to be merged! :smile: