about summary refs log tree commit diff stats
Commit message (Collapse)AuthorAgeFilesLines
* spawn: Use Popen23 for compatibilitytoonn2021-08-081-3/+6
|
* img_display: Use Popen23 for compatibilitytoonn2021-08-081-1/+2
|
* rifle: Drop with-statement when forkingtoonn2021-08-081-2/+1
| | | | | The Popen context manager wait()'s for the process to exit, which we don't want when forking.
* rifle: Switch to using Popen23toonn2021-08-081-5/+66
|
* popen_forked: Drop with-statementtoonn2021-08-081-5/+2
| | | | | The Popen context manager wait()'s for the process, which we don't want when forking it.
* popen23: Popen context manager wrappertoonn2021-08-081-0/+60
|
* py2_compat: Added check for with Popentoonn2021-08-081-1/+19
| | | | | Popen objects became context managers after Python 3.2 so we can't use them as such without a wrapper.
* Merge branch 'gh-actions'toonn2021-07-243-0/+12
|\
| * Actions: Add on pull_request againtoonn2021-07-243-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 branch 'osc52'toonn2021-07-241-2/+2
|\ \ | |/ |/|
| * commands: Yank, add missing mode to docstringtoonn2021-07-241-1/+1
| |
| * commands: Change precedence of clipboard managerstoonn2021-07-241-1/+1
|/ | | | | | | | | `wl-copy` used to take precedence over `xclip` and `xsel` but this can cause problems. So let's try the other way around, maybe Wayland users are less likely to have `xclip` and `xsel` installed than the other way around. Fixes #1861
* Merge remote-tracking branch 'EdwardBetts/spelling'toonn2021-07-249-11/+11
|\
| * Correct spellingEdward Betts2021-07-249-11/+11
|/
* Merge remote-tracking branch 'earboxer/fixgrep'toonn2021-07-221-1/+1
|\
| * grep command: fix to work with busybox grepZach DeCook2021-07-191-1/+1
| |
* | Merge branch 'gh-actions'toonn2021-07-205-43/+49
|\ \
| * | Actions: Check whether on_pullrequest is necessarytoonn2021-07-203-12/+0
| | |
| * | Actions: Reduce tested versions to oldest and newesttoonn2021-07-203-4/+4
| | |
| * | Actions: Fix Python and Pypy paths filterstoonn2021-07-202-3/+3
| | |
| * | Actions: Drop on_pullrequest from Pypytoonn2021-07-201-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 filtertoonn2021-07-202-4/+4
| | |
| * | Actions: Drop Pypy from pylint and add pylint to Pypytoonn2021-07-202-2/+2
| | |
| * | Actions: Update shellcheck required actionstoonn2021-07-201-3/+1
| | |
| * | Actions: Refine Pypy versions and titletoonn2021-07-201-2/+2
| | |
| * | Actions: Separate pypy tests because they failtoonn2021-07-203-2/+36
| | |
| * | Actions: Update, incorporate and splittoonn2021-07-203-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 pylinttoonn2021-07-201-1/+1
| | |
| * | Actions: Set TERM=dumb to avoid tput warningstoonn2021-07-202-0/+4
| | |
| * | Actions: Try to fix missing terminfotoonn2021-07-201-0/+1
| | |
| * | Goodbye Travis, don't let the door hit you on the way out!toonn2021-07-201-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 checkstoonn2021-07-203-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'toonn2021-07-2020-159/+239
|\ \ | |/ |/|
| * tests.manpage_completion: Use withtoonn2021-07-202-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 lentoonn2021-07-201-1/+1
| |
| * config.commands: Refactor to use withtoonn2021-07-202-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 withtoonn2021-07-201-6/+8
| |
| * gui.widgets.console: Refactor to use withtoonn2021-07-201-21/+24
| |
| * gui.colorscheme: Use with for file creationtoonn2021-07-201-1/+3
| |
| * ext.vcs.vcs: Disable frivolous lintstoonn2021-07-201-0/+2
| |
| * ext.spawn: Refactor to use withtoonn2021-07-201-4/+4
| |
| * ext.popen_forked: Refactor to use withtoonn2021-07-201-3/+6
| | | | | | | | Because the process forks we can closed the fds on this side.
| * ext.macrodict: Disable lint because of py2 compattoonn2021-07-201-1/+1
| |
| * ext.img_display: Disable lint for peculiar use of opentoonn2021-07-201-0/+4
| |
| * ext.accumulator: Use max rather than branchtoonn2021-07-201-2/+1
| |
| * core.runner: Disable lints because of peculiar uses of opentoonn2021-07-201-0/+3
| |
| * core.main: Refactor to use open23 and apply sensible lintstoonn2021-07-201-10/+10
| |
| * loader: Disable lint because of peculiar uses of opentoonn2021-07-201-1/+5
| |
| * actions: Disable lints because of peculiar use of opentoonn2021-07-201-0/+9
| |
| * tags: Refactor version conditional opentoonn2021-07-202-14/+27
| | | | | | | | | | Moving the open contextmanager that drops py3 keywords for py2 open to its own module so it can be reused.