Page MenuHomeSoftware Heritage

docs/developer-setup: Update documentation about auto-completion
ClosedPublic

Authored by ardumont on Feb 14 2019, 9:15 AM.

Details

Summary

Improving the postactivate hook example to work both for zsh/bash
users

Test Plan

After amending the postactivate hook described in the diff:

$ bash
$ workon swh
$ swh-scheduler <tab> <tab> # <- completion should work and show the scheduler's cli subcommands
$ deactivate
$ zsh
$ workon swh  # <- this no longer displays warnings/errors about completion
$ swh-scheduler <tab> <tab>  # <- completion should now work as in the bash shell

Diff Detail

Repository
rDDOC Development documentation
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 4213
Build 5557: arc lint + arc unit

Event Timeline

ardumont edited the test plan for this revision. (Show Details)
ardumont edited the test plan for this revision. (Show Details)
ardumont added inline comments.
docs/developer-setup.rst
79

i did the case/esac stuff as i found it clearer than the if/else
but also as kind of yagni for the day other shells start to get supported...

Fix indentation (missing 4 spaces)

This revision is now accepted and ready to land.Feb 14 2019, 9:32 AM
This revision was automatically updated to reflect the committed changes.