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

Jump to bottom

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

Merged
merged 1 commit into from Jul 27, 2021
Merged

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

merged 1 commit into from Jul 27, 2021

Conversation

irinaschubert
Copy link

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 July 27, 2021 06:57
@irinaschubert
Copy link
Author

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!

@irinaschubert
Copy link
Author

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?

BalduinLandolt
BalduinLandolt approved these changes Jul 27, 2021
View reviewed changes
knora/dsp_tools.py Show resolved Hide resolved
knora/dsp_tools.py Show resolved Hide resolved
@BalduinLandolt
Copy link
Collaborator

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.

@irinaschubert irinaschubert merged commit 9ecd794 into main Jul 27, 2021
@irinaschubert irinaschubert deleted the wip/DSP-1797-add-documentation-for-verbose-option branch July 27, 2021 07:31
irinaschubert pushed a commit that referenced this pull request Jul 27, 2021
fix(release): fix skipped release after pull request #74 (DSP-1797) (#76
c8e0a11 
)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BalduinLandolt BalduinLandolt BalduinLandolt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@irinaschubert @BalduinLandolt