-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
fix(appflowy_flutter): Flutter Version Reset Error #1923
fix(appflowy_flutter): Flutter Version Reset Error #1923
Conversation
- `flutter channel stable` was causing the FlutterSDK to upgrade to it's latest version. - Added code to fix this behaviour. - Tested on Windows and Linux.
Hi, @AmanNegi. The option AppFlowy : Setting up Flutter
grep: invalid option -- P
usage: grep [-abcdDEFGHhIiJLlMmnOopqRSsUVvwXxZz] [-A num] [-B num] [-C[num]]
[-e pattern] [-f file] [--binary-files=value] [--color=when]
[--context[=num]] [--directories=action] [--label] [--line-buffered]
[--null] [pattern] [file ...]
Unhandled exception: |
Hey @LucasXu0, I have fixed the issue. Could you please test it now, I don't have any |
Okay. Let me try it. |
- Replace `-P` with `-E` which should now work as expected in Mac devices.
- Fixed Commit Structure
82232c2
to
d4ab941
Compare
Hey, @AmanNegi. The script will complain as the below pic shows if the developer uses the other version of Flutter. Because the script switches to the Flutter path but doesn't switch back. |
Yep, thanks for pointing that out! I will fix that behavior. |
@LucasXu0 getting this weird error, any idea why this could be happening? I only made changes to the |
Now we move back to cwd after making changes
@LucasXu0 Could you please check if the changes I made, fixes the issue that we were facing? |
Hi, @AmanNegi. LGTM. And I changed the text format from CTLF to LF. |
flutter channel stable
was causing the FlutterSDK to upgrade to its latest version.Issue: #1912