-
-
Notifications
You must be signed in to change notification settings - Fork 250
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
feat(swift): mv to swift only support (no xcode tooling here) #670
Conversation
Review ChecklistDoes this PR follow the [Contribution Guidelines](development guidelines)? Following is a partial checklist: Proper conventional commit scoping:
|
damn I just realized it is not working. I tested it, but I forgot to remove my own configs first. My apologies. I'm on a fix. |
not sure it is the proper way, but this seems to work fine, and I pushed it. local utils = require "astronvim.utils"
return {
{
"nvim-treesitter/nvim-treesitter",
opts = function(_, opts)
if opts.ensure_installed ~= "all" then
opts.ensure_installed = utils.list_insert_unique(opts.ensure_installed, "swift")
end
end,
},
{
"williamboman/mason-lspconfig.nvim",
init = function() require("astronvim.utils.lsp").setup "sourcekit" end,
},
} |
resolves #669 |
@gacallea Added back codelldb. |
my bad. I thought it wasn't part of "basic" support. Good call. |
📑 Description
Moving away from any XCode tooling in Swift Lang pack. See: #669 (comment)
ℹ Additional Information
I am aware that opinionated is against the Astrocommunity guidelines. However, this pack has to be opinionated to make any sense. Otherwise the only thing in it would be treesitter. Not much to offer, really.
While not standardized (not for lack of trying), SwiftLint and SwiftFormat (not the Apple one) appear to be widely accepted by the community. They offer great customization too, in their own
.dotfiles
and it's an approach widely used by the community.I'd say that until Apple (or the community) sets on a standard, this is the best we could do.