Neslee Canil Pinto I've added you as a nearly full maintainer for this module, with the single exception of administering releases. Please contact me when you are ready to issue a release for me to review.

Comments

Jaypan created an issue. See original summary.

neslee canil pinto’s picture

@jaypan, ya ok sure 👍

jaypan’s picture

Hi Neslee - note that you can do releases die the other modules. Just not this one, as I’m trying to get it into core, so I’d like to review first.

neslee canil pinto’s picture

@Jaypan enabled Automated Testing for the Module.

1. #3029064: Typo fixes and applied Drupal coding standards - Reviewed and fixed the failing tests and committed the code to dev
2. #3152244: Automated Drupal 9 compatibility fixes - Reviewed and committed "Automated Drupal 9 compatibility fixes"
3. #3164396: Update source url in composer.json- Updated source url in composer.json
4. Added Neslee has a maintainer in composer.json for which i didnt create a issue.

And also for Color API module i have reviewed and committed the issues and created new issues too.

After your review and confirmation i will go forward with the new release for these 2 modules. Thanks

neslee canil pinto’s picture

Status: Active » Needs review
jaypan’s picture

Neslee - just to clarify, you mean a new D8 release is ready for the Vendor Stream Wrapper module?

neslee canil pinto’s picture

@jaypan, yes I did a review of deprecated code, coding standards and Drupal 9 compatible. And pushed the code to Dev branch

Once you are ok I will do Drupal 9 release

neslee canil pinto’s picture

@Jaypan did you get time to review the code commits?

neslee canil pinto’s picture

@Jaypan did you get any chance to review the commits?

podarok’s picture

Status: Needs review » Reviewed & tested by the community

Looks good. Let's release d9 version, please

neslee canil pinto’s picture

@podarok, @Jaypan had told that he needs to review the commits before we could release d9 version. Better we will wait for @Jaypan green signal😃

podarok’s picture

Fine, I'm using 1.x-dev version as of now in Open Y Distribution and all seems well in terms of installation https://github.com/ymcatwincities/openy/commit/9ffc0b61de29b3655fa85cea7...
Whenever the time is right - release it, please

neslee canil pinto’s picture

@podarok only thing pending is @Jaypan green signal to go. After that it's just to release a drupal 9 version.

podarok’s picture

Priority: Normal » Major

Seems like some delays. Can we proceed with the release of the module with Drupal 9 support? Dev version is tested and working well in Drupal 9.

neslee canil pinto’s picture

Sure, will cut a release this week.

podarok’s picture

Could you add me as a co-maintainer?
I'll work on Drupal 9 release.
Thank you

neslee canil pinto’s picture

Added, thanks for your help. I think you can directly go with a release as all the issues are fixed.

neslee canil pinto’s picture

Assigned: neslee canil pinto » Unassigned
Status: Reviewed & tested by the community » Fixed

The new release is out for drupal 8/9. Thank you.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.