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

docs(xmlupload): add documentation to verbose option (DSP-1797) #74

Merged

Conversation

@irinaschubert
Copy link
Collaborator

@irinaschubert irinaschubert commented Jul 27, 2021

resolves DSP-1797

@irinaschubert irinaschubert changed the title add documentation to verbose option, fix main method and path docs(verbose xmlupload): add documentation to verbose option (DSP-1797) Jul 27, 2021
@irinaschubert irinaschubert changed the title docs(verbose xmlupload): add documentation to verbose option (DSP-1797) docs(xmlupload): add documentation to verbose option (DSP-1797) Jul 27, 2021
@irinaschubert irinaschubert marked this pull request as ready for review Jul 27, 2021
@irinaschubert
Copy link
Collaborator Author

@irinaschubert irinaschubert commented Jul 27, 2021

Hallo @BalduinLandolt , es gab noch ein "module not found" error nach dem Deployment der letzten Version, weil die Zeile, in der der Pfad gesetzt wird, durch das "reformat code" verschoben wurde. Ausserdem gab's einen Fehler, weil die Main-Methode gefehlt hat. Ich hab sie deshalb wieder eingefügt... Und ausserdem noch Dokumentation ergänzt. Könntest du nochmals drüber schauen? Merci!

Loading

@irinaschubert
Copy link
Collaborator Author

@irinaschubert irinaschubert commented Jul 27, 2021

Hallo @BalduinLandolt , es gab noch ein "module not found" error nach dem Deployment der letzten Version, weil die Zeile, in der der Pfad gesetzt wird, durch das "reformat code" verschoben wurde. Ausserdem gab's einen Fehler, weil die Main-Methode gefehlt hat. Ich hab sie deshalb wieder eingefügt... Und ausserdem noch Dokumentation ergänzt. Könntest du nochmals drüber schauen? Merci!

Eigentlich seltsam, dass diese Fehler bei den Integration Tests nicht entdeckt wurden, oder? Dem sollten wir vllt. mal nachgehen, was meinst du?

Loading

knora/dsp_tools.py Show resolved Hide resolved
Loading
knora/dsp_tools.py Show resolved Hide resolved
Loading
@BalduinLandolt
Copy link
Collaborator

@BalduinLandolt BalduinLandolt commented Jul 27, 2021

Eigentlich seltsam, dass diese Fehler bei den Integration Tests nicht entdeckt wurden, oder? Dem sollten wir vllt. mal nachgehen, was meinst du?

ja... ich vermute, die Test hier sind noch durchaus verbesserungswürdig/-fähig.

Loading

@irinaschubert irinaschubert merged commit 9ecd794 into main Jul 27, 2021
4 of 5 checks passed
Loading
@irinaschubert irinaschubert deleted the wip/DSP-1797-add-documentation-for-verbose-option branch Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants