Skip to content

Conversation

@ralfdoering
Copy link
Contributor

The instructions missed the call to " (clhs:install-clhs-use-local)" and referenced the wrong directory for the .el file.

The instructions missed the call to " (clhs:install-clhs-use-local)" and referenced the wrong directory for the .el file.
@kilianmh kilianmh merged commit b934194 into LispCookbook:master Feb 20, 2023
@kilianmh
Copy link
Collaborator

Thank you for your contribution @ralfdoering!

@vindarel
Copy link
Contributor

Nice!

(note: I like that we "rebase + merge", more so with simple, self-contained commits, it prevents from seeing a merge commit, resulting in 2 commits for one change)

@kilianmh
Copy link
Collaborator

Actually, this was my first PR merging ;)
I thought it would be committed by default with only one commit. From now on it will be that way.

@vindarel
Copy link
Contributor

First PR merging on GitHub? Cool, welcome :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants