forked from maplibre/flutter-maplibre-gl
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Hello #6
Closed
Closed
Hello #6
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remi-deronzier
force-pushed
the
hello
branch
from
September 10, 2024 08:49
9bf87f2
to
868bce0
Compare
…`_addStylesheetToShadowRoot` (maplibre#409)
Co-authored-by: Peter Leibiger <kuhnroyal@gmail.com>
The main idea here is to get a the publishing working with the current state of the project. The `maplibre_gl_web` has never been published. With the new publishing workflow all packages will hopefully be published with a single tag ( `v0.19.0` in this case). * raise version to 0.19.0 * raise min Dart SDK to 3.0.0 as required by the `js: ^0.6.7` dependency * add `pubspec_overrides.yaml` files and move overridden local deps there * add `.pubignore` to prevent publishing unnecessary files * prepare new publish workflow * add supported platforms to `pubspec.yaml` * cancel in progress jobs if new commits are pushed * necessary lint fixes and formatting to satisfy Flutter version --------- Co-authored-by: Joscha <34318751+josxha@users.noreply.github.com>
Bumping version since we can not delete tags.
Since maplibre#390, the package can be used with Gradle 8. This is a small PR to have the example app use Gradle 8 for building (see maplibre#382) Tested with emulator and physical android device.
Since the example app uses gradle 8, the flutter beta ci fails because it still ises java 11. This pr bumps the version of java used to 17. Additionally it makes it possible to dispatch the workflow manually.
With the release to pub.dev, we can use the hosted API docs at https://pub.dev/documentation/maplibre_gl/latest/ and don't have to rely on our own generation. The current solution has the problem that the links on the right side bar linked to the raw html page. Co-authored-by: Peter Leibiger <kuhnroyal@gmail.com>
Co-authored-by: Peter Leibiger <kuhnroyal@gmail.com>
This pull request swaps out the basic flutter_lints ruleset for a more strict one, in this case the one from very_good_analysis because it gets used often in other projects and is well maintained. I made it that all packages reference the same lint file with the option to change the behavior for some if needed. - very_good_analysis has the same lower sdk bound as we, so it matches quite well. - I added all the lints to be ignored temporarily. Most of them are code smells, the use of dynamic typing, undocumented code or hints where code can be refactored to be more clean in general.
…#405) This pull request updates the maplibre-native version to `6.4.2` for iOS. Before the unstable version `5.14.0-pre3` was used. The iOS changelog for maplibre-native can be viewed [here](https://github.com/maplibre/maplibre-native/blob/main/platform/ios/CHANGELOG.md#600) (only iOS Devices with an Apple A7 GPU or later are supported from now on). Closes maplibre#388
…#405) This pull request updates the maplibre-native version to `6.4.2` for iOS. Before the unstable version `5.14.0-pre3` was used. The iOS changelog for maplibre-native can be viewed [here](https://github.com/maplibre/maplibre-native/blob/main/platform/ios/CHANGELOG.md#600) (only iOS Devices with an Apple A7 GPU or later are supported from now on). Closes maplibre#388
…e 2) (maplibre#406) This pull request updates the package to use version 11.0.0 of maplibre-native for android. It superseeds maplibre#403 to make it not dependant on other pull requests. If wanted, we can change the occurences of "Maplibre" to "MapLibre" to be more in line with maplibre-native. (I'd prefer to do it in a follow-up pr tho).
Not much to add here :)
We have currently a mix of `Maplibre` and `MapLibre`. This pull request changes all occurrences to `MapLibre` to be in line with the underlaying packages. I added deprecations for MaplibreMap, MaplibreMapController and MaplibreStyles to help users migrate.
Co-authored-by: Joscha <34318751+josxha@users.noreply.github.com>
Remi-deronzier
force-pushed
the
hello
branch
from
September 10, 2024 10:00
868bce0
to
1b4c8be
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.