Skip to content
This repository has been archived by the owner on Oct 25, 2018. It is now read-only.

Commit

Permalink
reproin: Don't hardcode HEAD~10 in afterlife solution
Browse files Browse the repository at this point in the history
Using HEAD~10 is fragile.  It needs to be updated if any of the steps
above are modified, and it assumes the solutions above were achieved
in exactly the same way, but it would be possible to get to the same
state with a different number of commits (e.g., by using 'datalad
download-url --path code/').
  • Loading branch information
kyleam committed Jun 12, 2018
1 parent d2214d1 commit 2030ddd
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions _episodes/03-01-reproin.md
Original file line number Diff line number Diff line change
Expand Up @@ -470,12 +470,14 @@ And because this record is complete, we can now simply throw away the input BIDS
> ## Task: Verify that the BIDS subdataset is unmodified and uninstall it
>
> Use the [datalad diff] command to check for modifications of the subdataset,
> and the [datalad uninstall] to delete it.
> Use the [datalad diff] command and `git log` to verify that the subdataset is
> in the same state as when it was initially added. Then use [datalad
> uninstall] to delete it.
>
> > ## Solution
> > ~~~
> > % datalad diff --revision HEAD~10 -- inputs
> > % datalad diff -- inputs
> > % git log -- inputs
> > % datalad uninstall --dataset . inputs --recursive
> > ~~~
> > {: .bash}
Expand Down

0 comments on commit 2030ddd

Please sign in to comment.