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.

Sign up for GitHub

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

Jump to bottom

Neues Modul: CSV Descriptor #110

Closed
wants to merge 6 commits into from
Closed

Neues Modul: CSV Descriptor #110

wants to merge 6 commits into from

Conversation

nevrrmind
Copy link

No description provided.

@nevrrmind nevrrmind changed the title Neues Plugin: CSV Descriptor Neues Modul: CSV Descriptor May 4, 2023
@Schrolli91 Schrolli91 linked an issue May 4, 2023 that may be closed by this pull request
Modul: CSV Descriptor #109
Open
@Schrolli91
Copy link
Member

Schrolli91 commented May 4, 2023 •
edited
Loading 

_______________ FLAKE8-check(ignoring E402 E501 E722 W504 W605) ________________
/home/runner/work/BW3-Core/BW3-Core/module/csv_descriptor.py:41:41: E231 missing whitespace after ':'
/home/runner/work/BW3-Core/BW3-Core/module/csv_descriptor.py:60:13: W292 no newline at end of file

@janspeller
Copy link
Member

2 Sachen fallen mir beim groben drüberherschauen auf:

  • Warum werden deutsche Variablennamen verwendet?
  • Warum ist ein Workflow für Pytest hinzugekommen?

Genaueres Review kann ich am Montag machen, bin momentan leider nicht zuhause

@nevrrmind
Copy link
Author

Die Variablen dienen mir einfach zur besseren Orientierung.
Die Sache mit dem Wokrflow hab ich nicht ganz verstanden und wohl mehr kopiert als nötig.
Mittlerweile hab ich das Modul weiter nach meinen Bedürfnissen angepasst und somit ist es für die breite Masse eher ungeeignet. Kann also ins Nirvana :)

@nevrrmind nevrrmind closed this May 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Schrolli91 Schrolli91 Awaiting requested review from Schrolli91

@janspeller janspeller Awaiting requested review from janspeller

Assignees
No one assigned
Labels
Component: Modul Feature Status: Review Needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Modul: CSV Descriptor
3 participants
@nevrrmind @Schrolli91 @janspeller