-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
I haven't checked yet, but I 'd prefer to have multiple but smaller PR's (for instance one per issue) |
Agreed. I will do this from now, but there were no issues created at the time and many modifications are very small like removing whitespace etc :) |
I'd suggest to first merge this one, before merging any others smaller merge requests that I add in the future. |
We already have a first global PR, now I want small PRs with clear title, clear description and link to issue. |
@MassimoC Could we still merge this one as a whole? It addresses a lot of smaller remarks by @fgheysels for which there is no issue. Splitting it up now would make it very messy. |
I referenced the related issues. |
we won't split up :) We will merge as soon reviewed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are normally returning problem+json for all the 4XX and 5XX.
This addresses issues
and also the other smaller remarks (see previous merge) by @fgheysels for which there are no seperate issues.
Note: The commits below also includes commits that were already in the previous merge request. I should have kept the commit tree cleaner.