fujin/thefuck
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
1,646 Commits 1 Branch 0 Tags
cf0921be4ad01fc644358a3e99680be5f238bfa7
Commit Graph

686 Commits

Author SHA1 Message Date
Joseph Frazier 892e8a8e65 Test parsing bash command substitution 
This is to help address bad corrections like the following (note the
position of the -p flag):

    thefuck 'git log $(git ls-files thefuck | grep python_command) -p'
    git log $(git ls-files thefuck | grep -p python_command) [enter/↑/↓/ctrl+c]
 2016-11-18 14:43:00 -05:00
Pablo Santiago Blum de Aguiar a947259eef #577: Use builtin history in Fish function 
Fix #577
 2016-11-17 22:57:11 -02:00
Vladimir Iakovlev aec8fe3233 #570: Refine tests 2016-11-08 23:53:40 +01:00
Vladimir Iakovlev c21dbd2be3 Merge branch 'git-flag-after-filename' of https://github.com/josephfrazier/thefuck into josephfrazier-git-flag-after-filename 2016-11-08 23:48:40 +01:00
Joseph Frazier 756044e087 Suggest ls -A when ls has no output 2016-10-31 13:49:38 -04:00
Joseph Frazier fa169c686c test_git_flag_after_filename.py: dedupe test commands 2016-10-31 00:22:24 -04:00
Joseph Frazier 9cae0bffff git_flag_after_filename: fix flake8 errors 
These were found by creating a `.flake8` file containing:

    [flake8]
    ignore = E501,W503
    exclude = venv

then running:

    flake8 $(git diff master... --name-only)

See https://github.com/nvbn/thefuck/pull/563 for running `flake8` in CI
 2016-10-31 00:22:24 -04:00
Joseph Frazier 5b420204c9 git: fix fatal: bad flag '...' after filename 
For example:

    $ git log README.md -p
    fatal: bad flag '-p' used after filename
    $ fuck
    git log -p README.md [enter/↑/↓/ctrl+c]
    Aborted

    $ git log -p README.md --name-only
    fatal: bad flag '--name-only' used after filename
    $ fuck
    git log -p --name-only README.md [enter/↑/↓/ctrl+c]
    Aborted

    $ git log README.md -p CONTRIBUTING.md
    fatal: bad flag '-p' used after filename
    $ fuck
    git log -p README.md CONTRIBUTING.md [enter/↑/↓/ctrl+c]
 2016-10-30 21:40:25 -04:00
Pablo Santiago Blum de Aguiar 07005b591a #N/A: Add git_rebase_merge_dir rule 2016-10-30 20:30:26 -02:00
Pablo Santiago Blum de Aguiar af28f0334a #N/A: Add git_rm_local_modifications rule 2016-10-29 17:51:55 -02:00
Joseph Frazier 521eb03d7a Fix flake8 errors: E127 continuation line over-indented for visual indent 2016-10-07 22:27:07 -04:00
Joseph Frazier e0cab4fa1b Fix flake8 errors: E126 continuation line over-indented for hanging indent 2016-10-07 22:27:07 -04:00
Joseph Frazier a9d55e9c62 Fix flake8 errors: E128 continuation line under-indented for visual indent 
See https://github.com/nvbn/thefuck/pull/563#discussion_r82492221
 2016-10-07 22:26:57 -04:00
Vladimir Iakovlev cf3acbfa2e Merge branch 'git-push-u' of https://github.com/josephfrazier/thefuck into josephfrazier-git-push-u 2016-10-07 10:40:02 +02:00
Vladimir Iakovlev 4d714994a3 Merge pull request #564 from josephfrazier/docker-python 
Use official Python images for Docker tests
 2016-10-07 10:38:49 +02:00
Vladimir Iakovlev 02f717a0e8 Merge pull request #562 from josephfrazier/man-help 
Suggest `foo --help` when `man foo` shows no pages
 2016-10-07 10:37:33 +02:00
Vladimir Iakovlev 8f4f2f03a7 Merge pull request #561 from josephfrazier/ag-literal 
Suggest `ag -Q` when relevant
 2016-10-07 10:35:34 +02:00
Joseph Frazier 432878bd77 Ignore flake8 errors with inline comments: W291 trailing whitespace 
See https://github.com/PyCQA/pycodestyle/pull/243
 2016-10-06 15:31:55 -04:00
Joseph Frazier 797b42cfd7 Fix flake8 errors: E302 expected 2 blank lines, found 1 2016-10-06 15:31:55 -04:00
Joseph Frazier 37161832aa Fix flake8 errors: E123 closing bracket does not match indentation of opening bracket's line 2016-10-06 15:31:55 -04:00
Joseph Frazier b221b04d0f Fix flake8 errors: F811 redefinition of unused... 2016-10-06 15:31:55 -04:00
Joseph Frazier dcc13bd2d2 Fix flake8 errors: F841, E265 
This commented-out test caused a couple flake8 errors, so get rid of it:
* F841 local variable 'cmd' is assigned to but never used
* E265 block comment should start with '# '

See https://github.com/nvbn/thefuck/pull/563#discussion_r82104360
 2016-10-06 15:30:53 -04:00
Joseph Frazier 283eb09c19 Fix flake8 errors: E231 missing whitespace after ',' 2016-10-06 13:30:39 -04:00
Joseph Frazier 10d409e6e2 Fix flake8 errors: E225 missing whitespace around operator 2016-10-06 13:30:13 -04:00
Joseph Frazier 93302c74b5 Fix flake8 errors: E101/W191 indentation contains (mixed spaces and) tabs 2016-10-06 13:30:13 -04:00
Joseph Frazier 22b005cebb Fix flake8 errors: E731 do not assign a lambda expression, use a def 2016-10-06 13:30:05 -04:00
Joseph Frazier feb36ede5c Fix suggestion for git push -u 
This was broken by https://github.com/nvbn/thefuck/pull/559
 2016-10-06 13:09:40 -04:00
Joseph Frazier 16a440cb9d test_zsh.py: use official python images, not ubuntu 
This should help reduce build times.
 2016-10-06 11:15:18 -04:00
Joseph Frazier 10b20574d1 test_tcsh.py: use official python images, not ubuntu 
This should help reduce build times.
 2016-10-06 11:12:38 -04:00
Joseph Frazier 91fceb401a test_fish.py: use official python images, not ubuntu 
This should help reduce build times.
 2016-10-06 11:09:19 -04:00
Joseph Frazier 4b79e23ba7 test_bash.py: use official python images, not ubuntu 
This should help reduce build times.
 2016-10-06 10:56:37 -04:00
Joseph Frazier f915a6ed0c test_performance.py: use python:3 image, not ubuntu 
This should help reduce build times.
 2016-10-06 10:54:47 -04:00
Joseph Frazier 77fc021a6c Refactor tests/rules/test_ag_literal.py 
https://github.com/nvbn/thefuck/pull/561#discussion_r81894710
 2016-10-05 10:52:24 -04:00
Joseph Frazier b2947aba8d test_ag_literal.py: Add blank line (PEP 8 E302) 
https://github.com/nvbn/thefuck/pull/561#discussion_r81892174
 2016-10-05 10:32:14 -04:00
Joseph Frazier 0c84eefa55 Don't suggest man 2/3 foo if no man pages exist 
Suggest `foo --help` instead. However, if there are man pages, suggest
`foo --help` after `man 2/3 foo`

This addresses the comment in the previous commit message:

> However, in cases where multiple sections have man pages for `foo`,
> running `man foo` could bring up the "wrong" section of man pages.
> `man read` is an example of this, but that should probably be handled in
> a way that still suggests `foo --help` first when there are *no* man
> pages for `foo` in any section.
 2016-10-03 14:10:42 -04:00
Joseph Frazier 8bd6c5da67 For man foo, try foo --help before man 3 foo 
`man` without a section searches all sections, so having `foo --help`
suggested first makes more sense than adding a specific section. See
https://github.com/nvbn/thefuck/pull/562#issuecomment-251142710

However, in cases where multiple sections have man pages for `foo`,
running `man foo` could bring up the "wrong" section of man pages.
`man read` is an example of this, but that should probably be handled in
a way that still suggests `foo --help` first when there are *no* man
pages for `foo` in any section.

Closes https://github.com/nvbn/thefuck/issues/546
 2016-10-03 12:03:57 -04:00
Joseph Frazier 5dbbb3b1ed Add ... --help to man suggestions 
This is along the lines of what @waldyrious suggested in
https://github.com/nvbn/thefuck/issues/546, but it just adds a new
suggestion rather than replacing the other ones.
 2016-10-03 03:57:53 -04:00
Joseph Frazier db4b37910d Suggest ag -Q when relevant 
This detects when `ag` suggests the `-Q` option, and adds it.
 2016-10-03 00:33:40 -04:00
Joseph Frazier 2b88ea11ea Suggest git diff --no-index when relevant 
This makes it easier to use `git diff` on untracked files.
 2016-10-03 00:05:01 -04:00
Vladimir Iakovlev db7dffdb44 Merge pull request #559 from josephfrazier/git-push-explicit-upstream 
Fix suggestions for `git push -u origin`
 2016-10-02 17:21:53 +02:00
Vladimir Iakovlev 7c4f0d2e55 Merge pull request #551 from scorphus/git-bisect-usage 
#N/A: Add `git_bisect_usage` rule
 2016-10-02 17:20:43 +02:00
Vladimir Iakovlev d05eb0a6dc #552: Fix code style 2016-10-02 17:19:33 +02:00
Vladimir Iakovlev cf352fd788 Merge branch 'remove-trailing-cedilla' of https://github.com/wikiti/thefuck into wikiti-remove-trailing-cedilla 2016-10-02 17:18:24 +02:00
Vladimir Iakovlev 3c1cce6bd2 Merge branch 'brew-link' of https://github.com/josephfrazier/thefuck into josephfrazier-brew-link 
# Conflicts:
#	README.md
 2016-10-02 17:17:15 +02:00
Vladimir Iakovlev 5d3a727d1a Merge pull request #555 from josephfrazier/brew-uninstall-force 
Suggest `brew uninstall --force` when relevant
 2016-10-02 17:14:54 +02:00
Vladimir Iakovlev ea87d55771 Merge pull request #554 from JordonPhillips/aws-rule 
Add new aws cli rule
 2016-10-02 17:14:14 +02:00
Joseph Frazier 934eeaf4fc Test that git push -u origin still works 
This was broken by https://github.com/nvbn/thefuck/pull/538
 2016-09-30 16:11:46 -04:00
Joseph Frazier bb5c7c576f Suggest brew link --overwrite --dry-run when relevant 
This makes it easier to see which files would be overwritten by
`brew link --overwrite`
 2016-09-30 15:31:25 -04:00
Joseph Frazier 17c3935078 Test brew uninstall --force suggestion 2016-09-29 17:44:07 -04:00
JordonPhillips 7bf405e9c3 Add aws cli rule 
This rule corrects spelling mistakes for aws cli commands and
subcommands.
 2016-09-29 14:22:08 -07:00