Skip to content
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

Documentar estándares de codificación esperados #144

Open
Scot3004 opened this issue Apr 27, 2018 · 4 comments
Open

Documentar estándares de codificación esperados #144

Scot3004 opened this issue Apr 27, 2018 · 4 comments
Labels
Docs Issues relacionados con documentación enhancement

Comments

@Scot3004
Copy link
Member

Incidente sitio web python Barranquilla

Comportamiento esperado

Definir en un documento, puede ser contributing o algún otro lineamientos relacionados con estándares de codificación esperados para el proyecto. Considerar el uso futuro de linters para validar que se cumplan dichos estándares.

Comportamiento actual

En el momento la documentación se limita a PEP8 sin embargo para el sitio web deben existir estándares para la codificación del sitio web

Pasos para reproducir el error

  1. Abrir el repositorio del proyecto
  2. Revisar archivo contributing.md
@GudarJs
Copy link
Collaborator

GudarJs commented Apr 27, 2018

Yo sugeriria usar https://prettier.io/ en lugar de instalar linters.

@Scot3004
Copy link
Member Author

Yo sugeriria usar https://prettier.io/ como uno de los linters a usar, ademas de eslint y stylelint

@Scot3004 Scot3004 added this to the Onboarding nuevos miembros milestone May 28, 2018
@Scot3004 Scot3004 added Docs Issues relacionados con documentación enhancement labels Oct 2, 2018
@Scot3004
Copy link
Member Author

Update, podemos usar husky + prettier, la verdad es que eslint no es muy necesario en este proyecto.

@Scot3004
Copy link
Member Author

Ejecute pylint en los archivos python actuales y encontre los siguientes errores

(venv) scot3004@SeCOrTo-V:~/proyectos/comunidades/django-quilla-web$ pylint --recursive=y tests scripts
************* Module tests.conftest
tests/conftest.py:11:0: E0401: Unable to import 'dotenv' (import-error)
tests/conftest.py:7:0: C0411: standard import "from typing import Generator" should be placed before "import pytest" (wrong-import-order)
************* Module scripts.events
scripts/events.py:6:0: C0301: Line too long (159/100) (line-too-long)
scripts/events.py:7:0: C0301: Line too long (137/100) (line-too-long)
scripts/events.py:15:50: C0303: Trailing whitespace (trailing-whitespace)
scripts/events.py:16:0: C0304: Final newline missing (missing-final-newline)
scripts/events.py:4:1: W0511: TODO: Create logic for oauth authentication when required (fixme)
scripts/events.py:1:0: C0114: Missing module docstring (missing-module-docstring)
scripts/events.py:6:22: W3101: Missing timeout argument for method 'requests.get' can cause your program to hang indefinitely (missing-timeout)
scripts/events.py:7:24: W3101: Missing timeout argument for method 'requests.get' can cause your program to hang indefinitely (missing-timeout)
scripts/events.py:15:5: W1514: Using open without explicitly specifying an encoding (unspecified-encoding)

-----------------------------------
Your code has been rated at 6.88/10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Issues relacionados con documentación enhancement
Projects
None yet
Development

No branches or pull requests

2 participants