Merged
Conversation
Occasionally the test runs on macos-13 are super slow and there is generally a lot of variation in the time required to run the test suite on those runners. I think they genuinely need more time sometimes, so this increases the timeout to accommodate.
* Run flake8 for all .py files tracked in git * Fix flake8 errors
this should keep it archived also when a server sets the threshold to a few years
* Add a codespell lint job * Fix misspellings found by codespell
* updated data base migration to current release * deprecation for manual migration added
`rtd.io` is on longer operational, according to [the project page](https://readthedocs.org/projects/mss/) the correct domain is either `rtfd.io` or the full `readthedocs.io`. Also switch to https.
* Fixed an issue with units being ignored in vertical axis * Fixed an issue with *assuming* that vertical axis is in pressure if no standard name "air_pressure" was available * Fixed an issue with extrapolating data * Fixes an issue with interpolating in pressure instead of log-pressure * Fixes an issue with ignoring jumps/extrapolation in data for missing longitudes (now it is consistent with vertical section driver)
* doc updated * indenting level fixed * add details about the wsgi modules
matrss
reviewed
Jul 4, 2024
Collaborator
There was a problem hiding this comment.
This PR mixes a bunch of unrelated changes, which makes it basically impossible to review for correctness of the intention of each change when applied to develop.
It was stated in #2332 that these PRs shouldn't be reviewed for the correctness of the intention of each commit, but instead just for if they are technically applicable to the target branch. But since I only find the former worthwhile for a review (after all, the latter is already done automatically by git through merge conflicts) I won't review this, as also stated in #2332.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose of PR?:
merge stable to develop