Closed (fixed)
Project:
Simple Google Maps
Version:
3.0.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
27 Jul 2022 at 11:57 UTC
Updated:
8 May 2023 at 11:13 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #3
nkoporecComment #4
nkoporecComment #6
nkoporecLast commit looks good, setting as RTBC.
Comment #7
phenaproximaFound a few small things.
Ideally we would also, like, run the tests on Drupal 9 and 10 before committing this, but I think that will require intervention from the maintainer.
Comment #8
martin107 commentedThank you all the for patches and discussion.
This patch is small enough so I can happily say "out of scope changes are welcome", plus locking down return types is really good.
This weekend
1) I will be running the tests.
2) We are altering "core_version_requirement", I am going to trigger a new 3.1 release.
If anyone has "internal business" rules and would like a 4.0 release ... just say the word.
Thank again
Comment #9
martin107 commentedAfter verifying this works on Drupal10
I have released version 4.
Thanks again
Comment #11
martin107 commentedAfter discussion with Phéna Proxima and Jakob Perry on slack ...
there is a discussion points about not bumping up to a 4.0.x release
but making the next release "3.1.0"
I am just leaving a breadcrumb to a summary of the reasons why...
https://medium.com/jakob-on-drupal/dont-go-making-major-version-changes-...
Comment #12
martin107 commentedOk, I am persuaded, thanks for the nudge
"Who this, new release."
Comment #13
anybodyCurrently there's no more release shown on the module page. Is this due to this issue?
The module is marked as "unsupported" at /admin/modules/update
Comment #14
2dareis2do commentedMAJOR version when you make incompatible API changes
MINOR version when you add functionality in a backwards compatible manner
PATCH version when you make backwards compatible bug fixes
https://semver.org
So from what I can see if you are dropping support for drupal 8, this is a major change.
Comment #15
2dareis2do commentedLooking forward to stable version of 4.x.x
Comment #16
martin107 commentedI have updated the home page
"3.0.1 Stable release covered by the Drupal Security Team released 5 June 2021
Works with Drupal: ^8 || ^9"
I hope that meets your needs.
Comment #17
2dareis2do commentedThis is what i am currently seeing:
Comment #18
luispe commentedSomewhere, where you specify the "supported_branches" of the project, it is wrong, it is marked as "3.1." and the latest version is 3.0.1, that's why it gives the (Unsupported error) in Drupal update process.
Comment #20
dishabhadra commented@martin107 Do you have any plan to release the Drupal 10-compatible version?