Patch or Minor release

  • Resolved Rune Rasmussen

    (@syntaxerrorno)


    5 months, 3 weeks ago

    Ref https://plugins.trac.www.ads-software.com/browser/svea-checkout-for-woocommerce/trunk/readme.txt#L59

    It’s clearly not a patch release when you add functionality and big rewrites, nor is a 2.8.0 version number indicating it’s a patch release according to SemVer – so even this correction is wrong. ??

    This version can anyhow not be used on anything bellow PHP 8.0.2, even if installed manually, because of the added dependencies (Composer), which probably also should have been mentioned in the change log – even if you have chosen to follow your own path for PHP requirements out of sync with Woo itself.

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author The Generation

    (@thegeneration)

    Hi,

    Thank you for noting the typo, we will update the next release!

    Regarding the PHP requirements we’ve already set the requirements to 8.0 as you mentioned so it’s not included in the patch notes.

    Best regards

    Thread Starter Rune Rasmussen

    (@syntaxerrorno)

    Thank you for noting the typo, we will update the next release!

    Such typos can have negative effects, when the updates are being flagged wrongly, so better ensure it not happens again.

    Regarding the PHP requirements we’ve already set the requirements to 8.0 as you mentioned so it’s not included in the patch notes.

    Yes, but you also have said it can be installed manually on lower PHP versions.

    Anyhow it was the addition of the required dependencies I had in mind, that you should have noted. ??

    Thread Starter Rune Rasmussen

    (@syntaxerrorno)

    Maybe you need a checklist to follow when releasing new versions?

    And also, don’t try to hide mistakes or other changes, always log everything. In this case, if someone manually updates (t.ex. by ftp), they should also know about this particular mistake, so they can delete the folder (even if it was out just a short time).

    Plugin Author The Generation

    (@thegeneration)

    Hi,

    We do have a checklist that we follow but we were a bit too fast this time. We’re only human.

    Thank you for pointing out the fact that the trunk directory got in the release as well. That wasn’t really a part of the release (hence not being in the change log) but a result of being quick with the next version. We’ll have a look at that release and perhaps remove it altogether since it cases issues.

    Best regards

Viewing 4 replies - 1 through 4 (of 4 total)
  • You must be logged in to reply to this topic.