Skip to content

feat(cli): add language server #74

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

feat(cli): add language server #74

wants to merge 1 commit into from

Conversation

azjezz
Copy link
Member

@azjezz azjezz commented Feb 8, 2025

πŸ“Œ What Does This PR Do?

πŸ” Context & Motivation

πŸ› οΈ Summary of Changes

  • Feature:
  • Bug Fix:
  • Refactor:
  • Docs:

πŸ“‚ Affected Areas

  • Linter
  • Formatter
  • CLI
  • Composer Plugin
  • Dependencies
  • Documentation
  • Other (please specify):

πŸ”— Related Issues or PRs

πŸ“ Notes for Reviewers

@azjezz azjezz added Experimental This feature or issue is experimental and may change or be removed in future versions. Priority: Low Can be picked up by anyone looking to contribute; a good entry-level fix. Status: On Hold Similar to Blocked, but currently assigned. Work is paused temporarily. Type: Enhancement Request for additions or changes that improve existing functionality. labels Feb 8, 2025
@azjezz azjezz self-assigned this Feb 8, 2025
@azjezz azjezz force-pushed the main branch 4 times, most recently from a8dd89b to 8c94f8c Compare February 21, 2025 01:50
JungoQuentin added a commit to JungoQuentin/mago that referenced this pull request May 20, 2025
from: carthage-software#74
Co-Authored-By: Saif Eddin Gmati <[email protected]>
@Bleksak
Copy link

Bleksak commented May 26, 2025

Hello, sorry if this is a wrong place to ask, but will this lsp eventually include completion, hover and auto includes? And how far is this from being completed? Thanks for the response.

@azjezz
Copy link
Member Author

azjezz commented May 27, 2025

Hi @Bleksak,

Yes, the 1.0.0 release of mago will include an LSP will all those features and more, however, it is not the current focus, we are currently working on the type checker, once that is stable, we will do a round of optimizations and refactor, then implement the LSP, which will include all those features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Experimental This feature or issue is experimental and may change or be removed in future versions. Priority: Low Can be picked up by anyone looking to contribute; a good entry-level fix. Status: On Hold Similar to Blocked, but currently assigned. Work is paused temporarily. Type: Enhancement Request for additions or changes that improve existing functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants