Skip to content

lazy#69

Merged
myuron merged 1 commit intomainfrom
refact
Apr 3, 2026
Merged

lazy#69
myuron merged 1 commit intomainfrom
refact

Conversation

@myuron
Copy link
Copy Markdown
Owner

@myuron myuron commented Apr 3, 2026

Summary by CodeRabbit

  • New Features

    • Added Terraform and HCL language support to syntax highlighting and parsing.
  • Bug Fixes

    • Updated Terraform language server configuration identifier.

@myuron myuron merged commit d4f58a6 into main Apr 3, 2026
0 of 2 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 3, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: aab519ed-e3d7-4790-9c9a-ca20fce07ff0

📥 Commits

Reviewing files that changed from the base of the PR and between 2e6b38c and fd91282.

📒 Files selected for processing (5)
  • nvim/lua/plugins/mini.lua
  • nvim/lua/plugins/nvim-cmp.lua
  • nvim/lua/plugins/nvim-dap.lua
  • nvim/lua/plugins/nvim-lspconfig.lua
  • nvim/lua/plugins/nvim-treesitter.lua

📝 Walkthrough

Walkthrough

Removed lazy = true flags from five Neovim plugin specifications (mini, nvim-cmp, nvim-dap, nvim-lspconfig, nvim-treesitter), changing their load behavior from explicitly lazy to default. Updated the Terraform LSP server identifier from "terraform_lsp" to "terraformls" and added "hcl" and "terraform" language parsers to Treesitter.

Changes

Cohort / File(s) Summary
Lazy Loading Removal
nvim/lua/plugins/mini.lua, nvim/lua/plugins/nvim-cmp.lua, nvim/lua/plugins/nvim-dap.lua
Removed lazy = true field from plugin specifications, shifting from explicit lazy loading to default load behavior.
LSP Configuration Update
nvim/lua/plugins/nvim-lspconfig.lua
Removed lazy = true field and updated LSP server identifier from "terraform_lsp" to "terraformls" in the enabled servers list.
Treesitter Enhancement
nvim/lua/plugins/nvim-treesitter.lua
Removed lazy = true field and extended language parser list by adding "hcl" and "terraform" to the installation configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • add:iri & nvim #28: Directly overlaps with edits to existing Neovim plugin specifications (mini, nvim-cmp, nvim-dap, nvim-lspconfig, nvim-treesitter).
  • add:cpp dap #50: Shares modifications to nvim-treesitter and nvim-lspconfig configurations, particularly language/server entry additions.

Poem

🐰 Lazy loading flags take flight,
Plugins eager, shining bright,
Terraform and HCL arrive,
Configurations come alive!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refact

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@myuron myuron deleted the refact branch April 3, 2026 11:01
@coderabbitai coderabbitai Bot mentioned this pull request Apr 24, 2026
Merged
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.

1 participant