-
Notifications
You must be signed in to change notification settings - Fork 14
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
Offline mode #51
base: main
Are you sure you want to change the base?
Offline mode #51
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy with the implementation besides the library addition.
I have a question about that below.
@@ -1,4 +1,5 @@ | |||
/// Stores all the commands used throughout the app that | |||
library; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why has this been added?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Auto added by linter.
linter: "Dangling library doc comment. Add a 'library' directive after the library comment."
has no effect other than for the auto generated API reference (not sure what the difference is though). Would change the above comment to "//" or keep library.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixes Stacked-Org/stacked#1140
Add offline_mode to stacked.json config to using stacked cli offline. It does so by passing the flag --no-pub to flutter run build_runner build. "offline mode" is of course just a suggestion easily changed by a maintainer or at the request of a maintainer.
This does pass tests apart from two which did not pass on the main branch either, I considered fixing them out of scope of this PR.