Skip to content

plugins/nvim-snippets: init#1979

Merged
mergify[bot] merged 2 commits intonix-community:mainfrom
psfloyd:nvim-snippets
Aug 7, 2024
Merged

plugins/nvim-snippets: init#1979
mergify[bot] merged 2 commits intonix-community:mainfrom
psfloyd:nvim-snippets

Conversation

@psfloyd
Copy link
Copy Markdown
Contributor

@psfloyd psfloyd commented Aug 3, 2024

Adds support for nvim-snippets.

Copy link
Copy Markdown
Member

@GaetanLepage GaetanLepage left a comment

Choose a reason for hiding this comment

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

Thanks for this contribution.
Your PR looks promising !

I made a few minor remarks.

Copy link
Copy Markdown
Member

@GaetanLepage GaetanLepage left a comment

Choose a reason for hiding this comment

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

mistake.

Copy link
Copy Markdown
Member

@GaetanLepage GaetanLepage left a comment

Choose a reason for hiding this comment

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

A bit more nit-picking...

@psfloyd psfloyd mentioned this pull request Aug 3, 2024
@psfloyd psfloyd requested a review from GaetanLepage August 5, 2024 15:11
Copy link
Copy Markdown
Member

@GaetanLepage GaetanLepage left a comment

Choose a reason for hiding this comment

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

This looks good to me !
@nix-community/nixvim could you please double-check quickly ?

@psfloyd, please squash your commits so as to have two left:

  • maintainers: add psfloyd
  • plugins/nvim-snippets: init

Copy link
Copy Markdown
Member

@MattSturgeon MattSturgeon left a comment

Choose a reason for hiding this comment

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

One nit, otherwise LGTM once other feedback is addressed!

@psfloyd
Copy link
Copy Markdown
Contributor Author

psfloyd commented Aug 7, 2024

It's ready to merge from my part. Squashed the commits and fixed the unnedded newline.

Copy link
Copy Markdown
Member

@MattSturgeon MattSturgeon left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Copy Markdown
Member

@GaetanLepage GaetanLepage left a comment

Choose a reason for hiding this comment

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

Well done, thanks !

@GaetanLepage
Copy link
Copy Markdown
Member

@Mergifyio queue

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Aug 7, 2024

queue

✅ The pull request has been merged automatically

Details

The pull request has been merged automatically at c9a855f

@mergify mergify bot merged commit c9a855f into nix-community:main Aug 7, 2024
@mergify mergify bot temporarily deployed to github-pages August 7, 2024 10:25 Inactive
@psfloyd psfloyd deleted the nvim-snippets branch August 8, 2024 23:23
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.

3 participants