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

New package : flutter-snippets #19

Merged
merged 6 commits into from
Jun 15, 2024
Merged

New package : flutter-snippets #19

merged 6 commits into from
Jun 15, 2024

Conversation

rishi-singh26
Copy link
Contributor

Package for some commonly used flutter code snippets.

@federico-terzi
Copy link
Contributor

Hey @rishi-singh26 ,

Sorry for the late reply! The package looks amazing, thank you for the help

Just one thing before we can merge, could you please:

  • Pull from main git pull origin main
  • Rename the flutter_snippets directory into flutter-snippets? :)
  • Repush into this branch

Cheers :)

Copy link
Collaborator

@smeech smeech left a comment

Choose a reason for hiding this comment

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

A simple text replacement package outputting impressively long and well-formatted code boilerplate.
No cmd:, script, shell, or malicious code, although I'm not familiar with Flutter.
Needs some tags: but then could safely be merged.

title: 'Flutter snippets'
description: A package for flutter code snippets
version: 0.1.0
author: rishi
Copy link
Collaborator

Choose a reason for hiding this comment

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

Some tags: would enable Espanso Hub to categorise the package, e.g. "Development", "Flutter", please.

@smeech smeech added the question Further information is requested label Jun 1, 2024
@smeech
Copy link
Collaborator

smeech commented Jun 15, 2024

No response from @rishi-singh26 to @federico-terzi or my comments.
Presumably abandoned by author.
I've renamed the directory from flutter_snippets to flutter-snippets.
An added some tags to _manifest.yml.

@smeech smeech merged commit 639a7fc into espanso:main Jun 15, 2024
1 check passed
@smeech smeech mentioned this pull request Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants