Skip to content
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

Fix: Fixes Misspelling In Diagnostics Settings Check #210

Merged
merged 15 commits into from
Mar 3, 2024
Merged

Conversation

harrisoncramer
Copy link
Owner

No description provided.

harrisoncramer and others added 15 commits February 26, 2024 08:00
* Fix: Toggle modified notes (#188)
* Fix: Toggle discussion nodes correctly
* Feat: Show Help keymap in discussion tree winbar

* Fix: Enable toggling nodes from the note body

Previously, the cursor had to be placed on the first line of the note
(the one with the author name) for toggling to work.

* Fix: Enable toggling resolved status from child nodes

Previously, the cursor had to be placed on the root node of the
discussion for toggling resolved status to work.

* Fix: Only try to show emoji popup on note nodes

Without this fix, there was an error whenever the cursor moved to a
file_name or path node in the "by_file_name" discussion tree.

* Feat: Add keymap for toggling tree type

Previously, tree type could only be set in the plugin configuration. Now
it can be toggled within a session. The keymap "i" is chosen based on a
similar functionality in DiffView.

* Fix: Disable tree type toggling in Notes

Tree type toggling only makes sense in the (linked) Discussions and
making it possible in the Notes could confuse users.
Fix: Multi-line discussions. The calculation of a range for a multiline comment has been consolidated and moved into the location.lua file. This does not attempt to fix diagnostics.
Refactor: It refactors the discussions code to split hunk parsing and management into a separate module
Fix: Don't allow comments on modified buffers #194 by preventing comments on the reviewer when using --imply-local and when the working tree is dirty entirely.
Refactor: It introduces a new List class for data aggregation, filtering, etc.
Fix: It removes redundant API calls and refreshes from the discussion pane
Fix: Allow users to switch tabs while creating comments
Fixes an issue in the old sha where the index when iterating through the lines of the hunk in the old SHA was off by one
* Updated lua/gitlab/actions/discussions/signs.lua

* Initial commit of changes

* Updated lua/gitlab/actions/discussions/signs.lua

* Updated lua/gitlab/actions/discussions/signs.lua

* Updated lua/gitlab/actions/discussions/init.lua

* Updated lua/gitlab/actions/discussions/signs.lua

* Split up signs and diagnostics

* Moved common to common file

* Continued modularization

* added formatting

* Moved signs/diagnostic lines into appropriate files

* Updated lua/gitlab/reviewer/init.lua

* Renamed is_current_sha => is_current_sha_focused

* Updated lua/gitlab/reviewer/init.lua

* Moved actions.indicators => indicators

* Split discussions/signs apart into separate modules

* Fixed filter function, fixed discussions

* Updated lua/gitlab/indicators/diagnostics.lua

* Setting diagnostics, handling and catching errors

* Greatly simplified diagnostics setup

* Re-split multi-line

* Updated lua/gitlab/indicators/diagnostics.lua

* Setting up signs

* Updated lua/gitlab/indicators/signs.lua

* Updated signs + display options

* Updated lua/gitlab/state.lua

* Updated doc/gitlab.nvim.txt

* Updated readme

* Added deprecation warning

* formatting
Fixes off-by one in new SHA comment creation, thank you @jakubbortlik for catching this
Fixes issue where jumping to the reviewer would jump to the wrong buffer (old SHA vs new SHA). Also addresses an issue in the refactor where ranged signs were being placed on top of comment signs.
Fixes typo in settings check, this is a #PATCH release
@harrisoncramer harrisoncramer changed the title Develop Fix: Fixes Misspelling In Diagnostics Settings Check Mar 3, 2024
@harrisoncramer harrisoncramer marked this pull request as ready for review March 3, 2024 17:15
@harrisoncramer harrisoncramer merged commit c73d229 into main Mar 3, 2024
6 checks passed
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