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

Unify XQuery and XQuery modules format and indentation #349

Merged
merged 114 commits into from
Feb 21, 2024

Conversation

bwbohl
Copy link
Member

@bwbohl bwbohl commented Feb 5, 2024

No description provided.

bwbohl and others added 4 commits February 15, 2024 01:01
fix wrong variable reference
remove trailing whitespace
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
@bwbohl
Copy link
Member Author

bwbohl commented Feb 21, 2024

@riedde I'm sorry to press a bit but any chance you complete the review of this by the end of the week?

add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
riedde
riedde previously approved these changes Feb 21, 2024
Copy link
Contributor

@riedde riedde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your work! Please have a look at my minor comments. After that it's approved :)

add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
add/data/xql/getSummary.xql Outdated Show resolved Hide resolved
bwbohl and others added 10 commits February 21, 2024 10:23
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
@bwbohl bwbohl requested a review from riedde February 21, 2024 09:30
riedde
riedde previously approved these changes Feb 21, 2024
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
bwbohl and others added 2 commits February 21, 2024 10:51
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
Co-authored-by: Dennis Ried <dennis.ried@uni-paderborn.de>
@bwbohl
Copy link
Member Author

bwbohl commented Feb 21, 2024

many thanks for reviewing @riedde ^^

@bwbohl bwbohl merged commit 90c3fb1 into develop Feb 21, 2024
1 check passed
@bwbohl bwbohl deleted the ftr/xquery-style-conformant branch February 21, 2024 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants