pghmcfc commented on the pull-request: `Add support for Cassandra` that you are 
following:
``
I don't think think this makes any difference to the resulting binary RPM. The 
Cassandra code is already packaged and is hence "supported" already. It does 
make the test suite run cleaner in that it doesn't skip 
`t/Apache-Session-Browseable-Cassandra.t` but that's just doing 
`use_ok('Apache::Session::Browseable::Cassandra')` rather than the extensive 
testing that's done for the equivalent redis test.

I think I'd prefer it if the changelog comment read something like "BR: 
perl(DBD::Cassandra) to improve test coverage".

Longer term, we might want to think about what to do with the existing hard 
dependency on `perl(Redis)` if Cassandra is a usable alternative.
``

To reply, visit the link below
https://src.fedoraproject.org/rpms/perl-Apache-Session-Browseable/pull-request/3
-- 
_______________________________________________
perl-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/[email protected]
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to