kompetenzinventar
/
ki-backend
7
6
Fork 7
Code Issues Pull Requests 2 Packages Projects Releases Activity

Add yapf to Pipenv environment #6

Merged
weeman merged 3 commits from add-yapf into main 2021-06-21 17:38:06 +02:00
Conversation 3 Commits 3 Files Changed 4 +17 -8
frlan commented 2021-06-21 16:11:35 +02:00
Owner
Copy Link

We should ensure that there is one stil of source code formating.

We should ensure that there is one stil of source code formating.
frlan added 3 commits 2021-06-21 16:11:35 +02:00
continuous-integration/drone/push Build is passing Details
continuous-integration/drone/pr Build is passing Details
325e680674 Reformat source code
weeman commented 2021-06-21 17:06:15 +02:00
Owner
Copy Link

Find ich gut. Ich würde auch später mal sehen Git-Hooks mit reinzunehmen.

Eine Frage hätte ich: Wenn yapf PEP8 formatiert und flake8 PEP8 validiert, wieso war es vorher schon gültig? Das sollte auf jeden Fall synchron sein.

Find ich gut. Ich würde auch später mal sehen Git-Hooks mit reinzunehmen. Eine Frage hätte ich: Wenn `yapf` PEP8 formatiert und flake8 PEP8 validiert, wieso war es vorher schon gültig? Das sollte auf jeden Fall synchron sein.
weeman requested changes 2021-06-21 17:06:40 +02:00
README.md Outdated
@ -43,6 +43,13 @@ python -m unittest discover ki
flake8
```
### Formating
weeman commented 2021-06-21 17:04:48 +02:00
Owner
Copy Link

Formating gibt es nicht. Entsprechend dem Rest der Datei Formatierung oder Formatieren?

`Formating` gibt es nicht. Entsprechend dem Rest der Datei `Formatierung` oder `Formatieren`?
frlan commented 2021-06-21 17:27:08 +02:00
Author
Owner
Copy Link

Pass ich an.

Pass ich an.
frlan commented 2021-06-21 17:28:24 +02:00
Author
Owner
Copy Link

Find ich gut. Ich würde auch später mal sehen Git-Hooks mit reinzunehmen.

Eine Frage hätte ich: Wenn yapf PEP8 formatiert und flake8 PEP8 validiert, wieso war es vorher schon gültig? Das sollte auf jeden Fall synchron sein.

pep8 gibt viele Möglichkeiten, wie code "richtig" formatiert sein kann. Tools wie yapf oder black sorgen dafür, dass es einheitlich nach einem Stil formatiert ist, welcher pep8 erfüllt. flake8 überprüft noch ein paar andere Sachen, wie z. B. ungenutzte Variablen.

> Find ich gut. Ich würde auch später mal sehen Git-Hooks mit reinzunehmen. > > Eine Frage hätte ich: Wenn `yapf` PEP8 formatiert und flake8 PEP8 validiert, wieso war es vorher schon gültig? Das sollte auf jeden Fall synchron sein. pep8 gibt viele Möglichkeiten, wie code "richtig" formatiert sein kann. Tools wie yapf oder black sorgen dafür, dass es einheitlich nach einem Stil formatiert ist, welcher pep8 erfüllt. flake8 überprüft noch ein paar andere Sachen, wie z. B. ungenutzte Variablen.
👍 1
frlan force-pushed add-yapf from 325e680674 to cbf3002b93 2021-06-21 17:29:09 +02:00 Compare
weeman merged commit ea7b6391c1 into main 2021-06-21 17:38:06 +02:00
weeman commented 2021-06-21 17:38:17 +02:00
Owner
Copy Link

Passt, danke für die Erläuterung.

Passt, danke für die Erläuterung.
Sign in to join this conversation.
Reviewers
No reviewers
weeman
Labels
Clear labels   
bug

Something is not working

  
doing

   
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
ready

Bereit für Release

  
review

   
wontfix

This won't be fixed

No Label
bug
doing
duplicate
enhancement
help wanted
invalid
question
ready
review
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: kompetenzinventar/ki-backend#6
No description provided.
Delete Branch "add-yapf"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?