Closed (fixed)
Project:
Vendor Stream Wrapper
Version:
8.x-1.x-dev
Component:
Miscellaneous
Priority:
Major
Category:
Plan
Assigned:
Unassigned
Reporter:
Created:
8 Aug 2020 at 19:41 UTC
Updated:
8 Nov 2020 at 17:44 UTC
Jump to comment: Most recent
Comments
Comment #2
neslee canil pinto@jaypan, ya ok sure 👍
Comment #3
jaypanHi 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.
Comment #4
neslee canil pinto@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
Comment #5
neslee canil pintoComment #6
jaypanNeslee - just to clarify, you mean a new D8 release is ready for the Vendor Stream Wrapper module?
Comment #7
neslee canil pinto@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
Comment #8
neslee canil pinto@Jaypan did you get time to review the code commits?
Comment #9
neslee canil pinto@Jaypan did you get any chance to review the commits?
Comment #10
podarokLooks good. Let's release d9 version, please
Comment #11
neslee canil pinto@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😃
Comment #12
podarokFine, 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
Comment #13
neslee canil pinto@podarok only thing pending is @Jaypan green signal to go. After that it's just to release a drupal 9 version.
Comment #14
podarokSeems 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.
Comment #15
neslee canil pintoSure, will cut a release this week.
Comment #16
podarokCould you add me as a co-maintainer?
I'll work on Drupal 9 release.
Thank you
Comment #17
neslee canil pintoAdded, thanks for your help. I think you can directly go with a release as all the issues are fixed.
Comment #18
neslee canil pintoThe new release is out for drupal 8/9. Thank you.