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

various: address diniamo's review comments #546

Merged
merged 3 commits into from
Jan 10, 2025
Merged

various: address diniamo's review comments #546

merged 3 commits into from
Jan 10, 2025

Conversation

NotAShelf
Copy link
Owner

And some other fixes. Not necessarily proud of the spellcheck one, but it beats taking an unchecked string.

@NotAShelf NotAShelf force-pushed the diniamo-moment branch 2 times, most recently from 66074fa to f95ef9c Compare January 10, 2025 09:32
@NotAShelf NotAShelf requested a review from diniamo January 10, 2025 10:01
modules/neovim/init/spellcheck.nix Outdated Show resolved Hide resolved
@NotAShelf NotAShelf requested a review from diniamo January 10, 2025 14:02
github-actions bot pushed a commit that referenced this pull request Jan 10, 2025
Copy link

github-actions bot commented Jan 10, 2025

🚀 Live preview deployed from 4df1cc3

View it here:

Debug Information

Triggered by: NotAShelf

HEAD at: diniamo-moment

Reruns: 33

@NotAShelf
Copy link
Owner Author

Hey the preview CI works, good to know.

@NotAShelf NotAShelf merged commit 4df1cc3 into main Jan 10, 2025
14 checks passed
Copy link

✅ Preview has been deleted successfully!

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.

2 participants