Skip to content
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

Flutter upgrade + Fix for #300 #303

Merged
merged 6 commits into from Mar 15, 2022

Conversation

davidrehakcz
Copy link
Contributor

  1. Upgrade Flutter - pouzita nejnovejsi verze 2.10.2 + definice min verze v pubspec.yaml
    --> tady bude zrejme nutne udelat upgrade flutter, jednotlivych pub modulu a taky Podu pro IOS vyvoj
  2. Fix pro Nepodporovaný typ příspěvku "log_message" #300

@kejml
Copy link
Contributor

kejml commented Feb 26, 2022

Pokud sis už nevšiml, build padá, protože v .travis.yml je zahardcodovana verze Flutteru, viz: https://github.com/lucien144/fyx/blob/develop/.travis.yml#L8

@davidrehakcz
Copy link
Contributor Author

Pokud sis už nevšiml, build padá, protože v .travis.yml je zahardcodovana verze Flutteru, viz: https://github.com/lucien144/fyx/blob/develop/.travis.yml#L8

Diky. Bude mi jeste nejaky cas trvat nez se ve vsem zorientuji. Vsechny podnety/komentare jsou urcite vitany.

@davidrehakcz
Copy link
Contributor Author

#300

image

@lucien144
Copy link
Owner

Za mě je to ready k mergnutí - vše funguje, zdá se, ok. Jen by mě zajímal ten multidex, zda ho nevyhodit...

@davidrehakcz
Copy link
Contributor Author

Za mě je to ready k mergnutí - vše funguje, zdá se, ok. Jen by mě zajímal ten multidex, zda ho nevyhodit...

Promin za pozdni reakci. To je samozrejme omyl a neni urcite treba, max pro vyvoj.

@kejml
Copy link
Contributor

kejml commented Mar 13, 2022

Laicky bych řekl, že ten multidex by vadit neměl. Vzhledem k tomu, že se to bez debug symobulů vejde do jednoho dex filu, tak to v něm zůstane a je to, ale je fakt, že ta knihovna může mít (hádám, že těžko měřitelný) overhead...

@lucien144 lucien144 merged commit 16db0b7 into lucien144:develop Mar 15, 2022
@davidrehakcz davidrehakcz deleted the flutter_upgrade branch March 16, 2022 19:23
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.

None yet

3 participants