-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add dart example #1191
base: main
Are you sure you want to change the base?
Add dart example #1191
Conversation
Nhan |
Nhab |
Trường Võ Văn Kiệt |
If you use # linux
FLUTTER_ROOT: /opt/hostedtoolcache/flutter/stable-3.10.6-x64
PUB_CACHE: /opt/hostedtoolcache/flutter/stable-3.10.6-x64/.pub-cache
# macos
FLUTTER_ROOT: /Users/runner/hostedtoolcache/flutter/stable-3.10.6-x64
PUB_CACHE: /Users/runner/hostedtoolcache/flutter/stable-3.10.6-x64/.pub-cache
# windows
FLUTTER_ROOT: C:\hostedtoolcache\windows\flutter\stable-3.10.6-x64
PUB_CACHE: C:\hostedtoolcache\windows\flutter\stable-3.10.6-x64/.pub-cache |
Wouldnt that be because flutter packages its own version of the dart sdk wouldnt it. That would definitely differ from just using dart. Not sure wether that needs to be specified or not. |
It's just an environment variable set by flutter-action (see the code snippet).
Since flutter and subosito/flutter-action are quite popular, maybe a separate Flutter section should be added alongside the Dart section? I feel like just having the dart section would cause some confusion |
yes, i also feel like there should be separate section for flutter but idk if this is allowed. if one of the maintainers replied maybe something could be done. |
Description
I've added an example for using the action within a dart repository
Motivation and Context
Faster workflows in any dart project.
How Has This Been Tested?
I've tested the same code snippet in the example in one of my projects and it ran as intended.
Screenshots (if appropriate):
Types of changes
Checklist: