Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update other-editors guide with modern vim references. #4089

Merged
merged 3 commits into from
Dec 23, 2021

Conversation

benknoble
Copy link
Contributor

After some Discord chatting, I realized this page was woefully out of date. Here are some attempts at updates.

/cc @spdegabrielle

@benknoble
Copy link
Contributor Author

I did not have a great time testing the docs locally, but there' s probably a better process. raco scribble --dest oe other-editors.scrbl && open oe/other-editors.html showed the red squigglies under #lang and racket and at-exp. I am probably missing for-label references. Not sure which ones to add.

@sorawee
Copy link
Collaborator

sorawee commented Dec 11, 2021

FWIW, this is what I see when building with raco setup scribblings/guide: 24.3 Vim.pdf

@benknoble
Copy link
Contributor Author

FWIW, this is what I see when building with raco setup scribblings/guide: 24.3 Vim.pdf

Great, that looks like the references were all correct then! 😄

@mflatt
Copy link
Member

mflatt commented Dec 13, 2021

Thanks for the update.

Nit: the capitalization of Vim looks inconsistent. Should it be Vim everywhere or vim everywhere?

@benknoble
Copy link
Contributor Author

Nit: the capitalization of Vim looks inconsistent. Should it be Vim everywhere or vim everywhere?

I… am not sure. I habitually write vim, but https://www.vim.org/ consistently writes Vim, so I need to correct both the document and the habit :)

@benknoble
Copy link
Contributor Author

@mflatt corrected. I also added a section on REPLs.

@mflatt mflatt merged commit cfeff58 into racket:master Dec 23, 2021
@benknoble benknoble deleted the update-vim-guide branch December 24, 2021 02:34
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.

None yet

3 participants