Skip to content

Comments

Feature: Add js plugin sample#1

Merged
jjw24 merged 2 commits intoFlow-Launcher:masterfrom
Zeroto521:js
Jan 26, 2021
Merged

Feature: Add js plugin sample#1
jjw24 merged 2 commits intoFlow-Launcher:masterfrom
Zeroto521:js

Conversation

@Zeroto521
Copy link
Member

Copy link
Member Author

@Zeroto521 Zeroto521 left a comment

Choose a reason for hiding this comment

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

Actually, the js developers are the most people in developers.

So, it's better to pull them to stand our side.

@Flow-Launcher/team

"Description": "Hello World Javascript",
"Author": "durland",
"Version": "1.0.0",
"Language": "executable",
Copy link
Member Author

Choose a reason for hiding this comment

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

The Flow needs to support the js not in the exe method to run the .js script.

@Zeroto521 Zeroto521 changed the title add js plugin sample Feature: Add js plugin sample Dec 18, 2020
@taooceros
Copy link
Member

Seems great, but why not make it native instead of running a bat file?

@jjw24
Copy link
Member

jjw24 commented Jan 26, 2021

Seems great, but why not make it native instead of running a bat file?

happy to merge this, we can do that later

@jjw24 jjw24 merged commit b066c1c into Flow-Launcher:master Jan 26, 2021
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