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

feat(integration): xcodebuild.nvim integration (#207) #212

Merged
merged 1 commit into from
Feb 6, 2024
Merged

feat(integration): xcodebuild.nvim integration (#207) #212

merged 1 commit into from
Feb 6, 2024

Conversation

wojciech-kulik
Copy link
Contributor

@wojciech-kulik wojciech-kulik commented Feb 5, 2024

Resolves #207

Also fixes off-by-one error in nvim-tree x_offset adjustment

@j-hui
Copy link
Owner

j-hui commented Feb 5, 2024

Oops, sorry I didn't see this. I already pushed d1b2a71 ; can you rebase on top of that? The comment you left in there is very helpful too.

@wojciech-kulik
Copy link
Contributor Author

Rebased :)

Copy link
Owner

@j-hui j-hui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks really great overall, thanks so much!

Just a couple of nits and questions, but honestly nothing I would consider blockers for the PR.

The main thing is I'd prefer either an explanation of why the nvim-tree offset was changed, or for that change to be reverted in both nvim-tree and xcodebuild.

lua/fidget/integration/xcodebuild-nvim.lua Outdated Show resolved Hide resolved
lua/fidget/integration/xcodebuild-nvim.lua Outdated Show resolved Hide resolved
lua/fidget/integration/nvim-tree.lua Show resolved Hide resolved
lua/fidget/integration/xcodebuild-nvim.lua Outdated Show resolved Hide resolved
@j-hui
Copy link
Owner

j-hui commented Feb 6, 2024

Oh, one more thing: can you drop a change in the README advertising this integration option? Here https://github.com/j-hui/fidget.nvim/blob/main/README.md?plain=1#L241-L245

@wojciech-kulik
Copy link
Contributor Author

Works quite well :)

demo.mov

@j-hui
Copy link
Owner

j-hui commented Feb 6, 2024

Looks awesome, I'm gonna merge it! Thanks so much for the contribution!

@j-hui j-hui merged commit 9eb2833 into j-hui:main Feb 6, 2024
1 check passed
@wojciech-kulik
Copy link
Contributor Author

🍻 thank you for the great plugin!

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.

feat: adjust fidget notification window offset when xcodebuild test drawer is open
2 participants