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

Document move to drevops namespace #87

Closed
xurizaemon opened this issue Jul 22, 2021 · 3 comments
Closed

Document move to drevops namespace #87

xurizaemon opened this issue Jul 22, 2021 · 3 comments

Comments

@xurizaemon
Copy link
Contributor

xurizaemon commented Jul 22, 2021

Following on from #76

Should we introduce some docs on any steps projects should to take for the update?

Context: I noticed this get picked up on a composer update --lock today and wondered if it might confuse people that the new package name is abandoned (see bottom of diff screenshot).

image

My expectation is that requiring the old name will continue to work, but if there's room for confusion or better guidance then happy to assist adding that. Maybe just a few words in the README's installation section to address it?

@xurizaemon
Copy link
Contributor Author

xurizaemon commented Jul 22, 2021

What I hadn't spotted was the output from composer update:

Package integratedexperts/behat-steps is abandoned, you should avoid using it. Use drevops/behat-steps instead.

That might be sufficient - but I think a note about the rename (and that it's the same package) might be useful in the README? Will PR something when I have a chance, lmk if you think unnecessary Alex :)

@AlexSkrypnyk
Copy link
Member

@xurizaemon
Thank you for your feedback
I was asking around in the community how to better implement this "change notification" and the response I got was that an abandonment message is sufficient enough.

The new name cannot work as Packagist does not allow to rename project namespace (it only allows to point to a GitHub package with new org namespace), so the whole project had to be re-created in Packagist.

What I will do now is add more information to release notes for 1.0.0.

Readme also have updated information on how to include traits (using new namespace DrevOps):

<?php

use Drupal\DrupalExtension\Context\DrupalContext;
use DrevOps\BehatSteps\D7\ContentTrait;

/**
 * Defines application features from the specific context.
 */
class FeatureContext extends DrupalContext {

  use ContentTrait;

}

@AlexSkrypnyk
Copy link
Member

Documented in https://github.com/drevops/behat-steps/releases/tag/1.0.0

please feel free to re-open

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

No branches or pull requests

2 participants