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

Better errors messages #3195

Closed
wants to merge 3 commits into from
Closed

Better errors messages #3195

wants to merge 3 commits into from

Conversation

sbrunner
Copy link
Member

@sbrunner sbrunner commented Feb 9, 2024 •
edited by geo-ghci-int bot
Loading

See JIRA issue: GSSGA-96.

@sbrunner sbrunner requested a review from sebr72 February 9, 2024 14:50
@sbrunner sbrunner force-pushed the better-errors-GSSGA-96 branch 2 times, most recently from 5c95d88 to d095ee9 Compare February 13, 2024 15:17
@sbrunner sbrunner removed the request for review from sebr72 February 14, 2024 16:40
@sbrunner sbrunner mentioned this pull request Apr 30, 2024
Merged
@sbrunner sbrunner closed this Aug 8, 2024
@sbrunner sbrunner reopened this Aug 8, 2024
@sbrunner sbrunner marked this pull request as ready for review August 8, 2024 12:06
@sbrunner sbrunner requested a review from sebr72 August 8, 2024 12:06
@mapfish mapfish deleted a comment from github-actions bot Aug 8, 2024
sebr72
sebr72 requested changes Aug 8, 2024
View reviewed changes
Copy link
Contributor

@sebr72 sebr72 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I removed backport master.
An improved implementation of these changes are in #3269 in Master. Maybe we should use it and apply to 3.30. Or we should copy the changes from #3269 into that one ?

@sbrunner
Copy link
Member Author

sbrunner commented Aug 8, 2024

Or just close this pull request :-)

@sbrunner sbrunner closed this Aug 8, 2024
@sbrunner sbrunner deleted the better-errors-GSSGA-96 branch August 8, 2024 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sebr72 sebr72 sebr72 requested changes

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

Successfully merging this pull request may close these issues.
2 participants
@sbrunner @sebr72