pub-solar/os
5
0
Fork 2
Code Issues 4 Pull requests Packages Projects 1 Releases Wiki Activity

feat: use nnn and treesitter #235

Merged
teutat3s merged 7 commits from feat/nnn into main 2023-07-21 09:46:27 +00:00
Conversation 0 Commits 7 Files changed 13 +178 -585
b12f commented 2023-07-13 16:08:35 +00:00
Owner
Copy link

vifm is removed in favour of nnn, with the keybindings in nvim being
changed. TODO: improve the colorscheme of nnn to match ls.

All nvim plugins that were responsible for language syntax highlighting
are removed in favour of treesitter[1], a syntax highlighter that parses
AST instead of being regex-based. This is still experimental, but my
tests so far have not encountered any issues.

  1. https://github.com/nvim-treesitter/nvim-treesitter
vifm is removed in favour of nnn, with the keybindings in nvim being changed. TODO: improve the colorscheme of `nnn` to match `ls`. All nvim plugins that were responsible for language syntax highlighting are removed in favour of treesitter[1], a syntax highlighter that parses AST instead of being regex-based. This is still experimental, but my tests so far have not encountered any issues. 1) https://github.com/nvim-treesitter/nvim-treesitter
b12f added 2 commits 2023-07-13 16:08:37 +00:00
feat: use nnn and treesitter
All checks were successful
continuous-integration/drone/push Build is passing
Details
41c2d668f7 
vifm is removed in favour of nnn, with the keybindings in nvim being
changed. TODO: improve the colorscheme of `nnn` to match `ls`.

All nvim plugins that were responsible for language syntax highlighting
are removed in favour of treesitter[1], a syntax highlighter that parses
AST instead of being regex-based. This is still experimental, but my
tests so far have not encountered any issues.

1) https://github.com/nvim-treesitter/nvim-treesitter
fix: load schemastore for nvim
Some checks failed
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is failing
Details
e36709613c
teutat3s added 2 commits 2023-07-14 11:13:33 +00:00 
• Updated input 'latest':
    'github:nixos/nixpkgs/645ff62e09d294a30de823cb568e9c6d68e92606' (2023-07-01)
  → 'github:nixos/nixpkgs/2de8efefb6ce7f5e4e75bdf57376a96555986841' (2023-07-12)
• Updated input 'nixos':
    'github:nixos/nixpkgs/b72aa95f7f096382bff3aea5f8fde645bca07422' (2023-06-30)
  → 'github:nixos/nixpkgs/fcc147b1e9358a8386b2c4368bd928e1f63a7df2' (2023-07-13)
Fix merge conflict leftovers
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details
9271e8d037
teutat3s added 1 commit 2023-07-14 11:40:30 +00:00
neovim: add syntax highlighting for ini, lua,
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details
1fe6d34f6e 
bash, make, ninja, diff, php

Sort nvim-treesitter.withPlugins alphabetically
teutat3s reviewed 2023-07-14 11:50:34 +00:00
profiles/base-user/.config/git/gitmessage.nix Outdated
@ -8,3 +8,2 @@
in ''
# Title: Summary, imperative, start upper case, don't end with a period
# No more than 50 chars. #### 50 chars is here: #
# What happened?
teutat3s commented 2023-07-14 11:48:56 +00:00
Owner
Copy link

Suggestion:

# Title: What happened? (summary, imperative)
# No more than 50 chars. #### 50 chars is here:  #
Suggestion: ``` # Title: What happened? (summary, imperative) # No more than 50 chars. #### 50 chars is here: # ```
b12f marked this conversation as resolved
profiles/base-user/.config/git/gitmessage.nix Outdated
@ -11,0 +15,4 @@
# ^\n
# What exactly was done and why?
teutat3s commented 2023-07-14 11:46:47 +00:00
Owner
Copy link

Suggestion:

# ^\n
# Description: What exactly was done and why?
# Wrap at 72 chars. ################################## which is here:  #
Suggestion: ``` # ^\n # Description: What exactly was done and why? # Wrap at 72 chars. ################################## which is here: # ```
b12f marked this conversation as resolved
profiles/base-user/.config/git/gitmessage.nix Outdated
@ -16,1 +22,3 @@
# Wrap at 72 chars. ################################## which is here: #
# ^\n
#
# Any issue numbers or links?
teutat3s commented 2023-07-14 11:50:29 +00:00
Owner
Copy link

Suggestion:

# Any related issue numbers or links?
Suggestion: ``` # Any related issue numbers or links? ```
b12f marked this conversation as resolved
teutat3s reviewed 2023-07-14 11:52:51 +00:00
modules/terminal-life/nvim/default.nix Outdated
@ -65,3 +128,2 @@
# Get sudo in vim; :SudaWrite <optional filename>
suda-vim
syntastic
vim-gutentags
teutat3s commented 2023-07-14 11:52:51 +00:00
Owner
Copy link

Is removing vim-gutentags intentional? I remember we still have these tags files lying around in every repo. In case it is, let's clean up all mentions of tags as well.

Is removing `vim-gutentags` intentional? I remember we still have these `tags` files lying around in every repo. In case it is, let's clean up all mentions of tags as well.
b12f marked this conversation as resolved
teutat3s reviewed 2023-07-14 11:54:27 +00:00
profiles/base-user/.config/git/gitmessage.nix Outdated
@ -11,0 +10,4 @@
#
# fix feat build chore ci docs style refactor perf test
#
# type!(optional scope): <description>
teutat3s commented 2023-07-14 11:54:27 +00:00
Owner
Copy link

Suggestion:

# type!(optional scope): <summary>
Suggestion: ``` # type!(optional scope): <summary> ```
b12f marked this conversation as resolved
b12f added 2 commits 2023-07-20 22:00:02 +00:00
feat: add line length indicators to gitmessage
All checks were successful
continuous-integration/drone/push Build is passing
Details
continuous-integration/drone/pr Build is passing
Details
9bade4eb64
teutat3s approved these changes 2023-07-21 09:45:51 +00:00
teutat3s left a comment
Owner
Copy link

Thanks! 🚢 it!

Thanks! :ship: it!
teutat3s merged commit 8f4f5687f2 into main 2023-07-21 09:46:27 +00:00
teutat3s deleted branch feat/nnn 2023-07-21 09:46:27 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
teutat3s
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: pub-solar/os#235
No description provided.
Delete branch "feat/nnn"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?