Skip to content

Conversation

jahidulpabelislam
Copy link
Contributor

@jahidulpabelislam jahidulpabelislam commented Jul 24, 2025

It is supported to have -dev as a suffix in composer, which pretty much behaves same as a prefix (dev-) - see https://getcomposer.org/doc/articles/versions.md#branches.. however if you do do this this package doesn't pick up this as a diff, so I've updated the if to make it supported.

@jahidulpabelislam
Copy link
Contributor Author

@SmetDenis Any chance you could have a look at this pls.

@jahidulpabelislam
Copy link
Contributor Author

@SmetDenis I've added a unittest for it.

@SmetDenis
Copy link
Member

@jahidulpabelislam
Thanks for the PR, this might be useful. Can you tell me, please, what problem this solves?

@jahidulpabelislam
Copy link
Contributor Author

@SmetDenis does the pr description not provide enough info ?

Basically im not seeing a package show up in list of diff when it should.

@jahidulpabelislam
Copy link
Contributor Author

@SmetDenis Can I nudge on this please. It is quite a simple change.

@SmetDenis SmetDenis merged commit 6c7b5f3 into JBZoo:master Sep 28, 2025
1 check passed
@SmetDenis
Copy link
Member

SmetDenis commented Sep 28, 2025

@jahidulpabelislam hello.
Apologies for the delayed response. I was caught up with many other thinkgs and completely forgot about you.

I've released the new version 7.1, including your changes.

Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants