-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
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