Skip to content

Conversation

@themilkman
Copy link
Contributor

Hello everyone,

I just updated the docs regarding Sublime Text plugins and LSP. Hope this helps other users.

Please feel free to request changes or do them on your own if I missed something. Thanks!

Cheers!


Before submitting the PR make sure the following are checked:

  • [x ] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
  • [ x] Wrote good commit messages.
  • [ -] Commit message starts with [Fix #issue-number] (if the related issue exists).
  • [x ] Feature branch is up-to-date with master (if not - rebase it).
  • Squashed related commits together.
  • [ -] Added tests.
  • [- ] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
  • Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

@koic
Copy link
Member

koic commented Nov 14, 2023

Thank you for updating the docs! Can you incorporate the comments I left and squash your commits into one?

The already mentioned plugin is not actively maintained anymore, thus
added the well maintained SublimeLinter-rubocop plugin.
Also added docs on LSP usage with Sublime Text.
@themilkman themilkman force-pushed the sublime_text_doc_extensions branch from ae1481a to 7a75681 Compare November 14, 2023 14:57
@themilkman
Copy link
Contributor Author

Hi @koic, thanks for your super fast feedback!

I applied the suggested changes and squashed them. Hope it is fine now, otherwise please let me know.

@koic koic merged commit 37fe32d into rubocop:master Nov 14, 2023
@koic
Copy link
Member

koic commented Nov 14, 2023

Thanks!

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