Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Actions: Add on pull_request again | toonn | 2021-07-24 | 3 | -0/+12 |
| | | | | | | | | | | | | | | | My testing was flawed. The status checks seemed to work fine because the builds were triggered on push in the same repo. But this doesn't work across repos. So GH Actions having run to completion on someone's fork doesn't give us the status checks we need. Therefore we want to run our Actions on PRs too, that way they're run from the ranger repo as well and the status checks should actually be informative. One possible improvement is that we could duplicate the workflows we want to run on PRs, that way we can drop all the things that don't really matter for status checks from the version matrices. Otoh that means we have pretty much duplicated workflows we need to keep in sync. Thank you for your lack of abstractions, GH Actions 🙄 | ||||
* | Merge remote-tracking branch 'EdwardBetts/spelling' | toonn | 2021-07-24 | 9 | -11/+11 |
|\ | |||||
| * | Correct spelling | Edward Betts | 2021-07-24 | 9 | -11/+11 |
|/ | |||||
* | Merge remote-tracking branch 'earboxer/fixgrep' | toonn | 2021-07-22 | 1 | -1/+1 |
|\ | |||||
| * | grep command: fix to work with busybox grep | Zach DeCook | 2021-07-19 | 1 | -1/+1 |
| | | |||||
* | | Merge branch 'gh-actions' | toonn | 2021-07-20 | 5 | -43/+49 |
|\ \ | |||||
| * | | Actions: Check whether on_pullrequest is necessary | toonn | 2021-07-20 | 3 | -12/+0 |
| | | | |||||
| * | | Actions: Reduce tested versions to oldest and newest | toonn | 2021-07-20 | 3 | -4/+4 |
| | | | |||||
| * | | Actions: Fix Python and Pypy paths filters | toonn | 2021-07-20 | 2 | -3/+3 |
| | | | |||||
| * | | Actions: Drop on_pullrequest from Pypy | toonn | 2021-07-20 | 1 | -6/+2 |
| | | | | | | | | | | | | | | | We need to get it passing first before we can make it a useful status check on PRs. | ||||
| * | | Actions: Fix pylint and Pypy paths filter | toonn | 2021-07-20 | 2 | -4/+4 |
| | | | |||||
| * | | Actions: Drop Pypy from pylint and add pylint to Pypy | toonn | 2021-07-20 | 2 | -2/+2 |
| | | | |||||
| * | | Actions: Update shellcheck required actions | toonn | 2021-07-20 | 1 | -3/+1 |
| | | | |||||
| * | | Actions: Refine Pypy versions and title | toonn | 2021-07-20 | 1 | -2/+2 |
| | | | |||||
| * | | Actions: Separate pypy tests because they fail | toonn | 2021-07-20 | 3 | -2/+36 |
| | | | |||||
| * | | Actions: Update, incorporate and split | toonn | 2021-07-20 | 3 | -41/+38 |
| | | | | | | | | | | | | | | | | | | Update the required actions. Incorporate doctest and manpage_completion test in the test action and separate pylint because it doesn't work with as many python versions. | ||||
| * | | Actions: Drop 3.5 from linting because of old pylint | toonn | 2021-07-20 | 1 | -1/+1 |
| | | | |||||
| * | | Actions: Set TERM=dumb to avoid tput warnings | toonn | 2021-07-20 | 2 | -0/+4 |
| | | | |||||
| * | | Actions: Try to fix missing terminfo | toonn | 2021-07-20 | 1 | -0/+1 |
| | | | |||||
| * | | Goodbye Travis, don't let the door hit you on the way out! | toonn | 2021-07-20 | 1 | -20/+0 |
| | | | | | | | | | | | | | | | | | | Travis CI has stopped supporting open source projects without jumping through hoops. We're giving up on them in favor of other alternatives, GH Actions and potentially Circle CI. | ||||
| * | | Actions: Add on pull_request to get status checks | toonn | 2021-07-20 | 3 | -3/+11 |
| |/ | | | | | | | | | | | This should enable the pylint, pytest, flake8 and shellcheck lints to run when PRs are opened and have them act as status checks so PRs aren't merged without them passing. | ||||
* | | Merge branch 'pylint3k' | toonn | 2021-07-20 | 20 | -159/+239 |
|\ \ | |/ |/| | |||||
| * | tests.manpage_completion: Use with | toonn | 2021-07-20 | 2 | -2/+3 |
| | | | | | | | | | | This test revealed an exception that wasn't covered in `ranger/__init__.py` due to the refactor to a with-statement. | ||||
| * | container.history: Fix type error due to forgotten len | toonn | 2021-07-20 | 1 | -1/+1 |
| | | |||||
| * | config.commands: Refactor to use with | toonn | 2021-07-20 | 2 | -16/+42 |
| | | | | | | | | | | This required adding another argument to `open23` so I just implemented all the arguments the open builtin has. | ||||
| * | ranger.__init__: Refactor to use with | toonn | 2021-07-20 | 1 | -6/+8 |
| | | |||||
| * | gui.widgets.console: Refactor to use with | toonn | 2021-07-20 | 1 | -21/+24 |
| | | |||||
| * | gui.colorscheme: Use with for file creation | toonn | 2021-07-20 | 1 | -1/+3 |
| | | |||||
| * | ext.vcs.vcs: Disable frivolous lints | toonn | 2021-07-20 | 1 | -0/+2 |
| | | |||||
| * | ext.spawn: Refactor to use with | toonn | 2021-07-20 | 1 | -4/+4 |
| | | |||||
| * | ext.popen_forked: Refactor to use with | toonn | 2021-07-20 | 1 | -3/+6 |
| | | | | | | | | Because the process forks we can closed the fds on this side. | ||||
| * | ext.macrodict: Disable lint because of py2 compat | toonn | 2021-07-20 | 1 | -1/+1 |
| | | |||||
| * | ext.img_display: Disable lint for peculiar use of open | toonn | 2021-07-20 | 1 | -0/+4 |
| | | |||||
| * | ext.accumulator: Use max rather than branch | toonn | 2021-07-20 | 1 | -2/+1 |
| | | |||||
| * | core.runner: Disable lints because of peculiar uses of open | toonn | 2021-07-20 | 1 | -0/+3 |
| | | |||||
| * | core.main: Refactor to use open23 and apply sensible lints | toonn | 2021-07-20 | 1 | -10/+10 |
| | | |||||
| * | loader: Disable lint because of peculiar uses of open | toonn | 2021-07-20 | 1 | -1/+5 |
| | | |||||
| * | actions: Disable lints because of peculiar use of open | toonn | 2021-07-20 | 1 | -0/+9 |
| | | |||||
| * | tags: Refactor version conditional open | toonn | 2021-07-20 | 2 | -14/+27 |
| | | | | | | | | | | Moving the open contextmanager that drops py3 keywords for py2 open to its own module so it can be reused. | ||||
| * | tags: Use with where possible disable frivolous lint | toonn | 2021-07-20 | 1 | -8/+14 |
| | | |||||
| * | history: Replace branching with min/max builtins | toonn | 2021-07-20 | 1 | -8/+2 |
| | | |||||
| * | bookmarks: Switch to using with | toonn | 2021-07-20 | 1 | -14/+13 |
| | | |||||
| * | img_display: Drop redundant close calls | toonn | 2021-07-15 | 1 | -3/+0 |
| | | |||||
| * | img_display: Use with where possible | toonn | 2021-07-15 | 1 | -34/+37 |
| | | |||||
| * | rifle.py: Pylint insists on no bare Popen()s | toonn | 2021-07-15 | 1 | -1/+2 |
| | | |||||
| * | rifle.py: Using with where possible | toonn | 2021-07-15 | 1 | -26/+35 |
|/ | |||||
* | actions.py: Another implicit return | toonn | 2021-07-15 | 1 | -1/+1 |
| | |||||
* | actions.py: Simplify a boolean condition to appease pylint | toonn | 2021-07-15 | 1 | -1/+1 |
| | |||||
* | commands.py: Pylint doesn't like mixing im/explicit returns | toonn | 2021-07-15 | 1 | -3/+6 |
| | |||||
* | Merge branch 'clarify-imgdisplayunsupported' | toonn | 2021-07-14 | 1 | -3/+9 |
|\ |