[Bug 1979724] Review Request: python-applicationinsights - Python support for Azure Application Insights API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1979724

Major Hayden 🤠 <mhayden@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|needinfo?(mhayden@xxxxxxxxx |
                   |m)                          |



--- Comment #2 from Major Hayden 🤠 <mhayden@xxxxxxxxxx> ---
So the %{python3_pkgversion} was in my local copy, but I guess I built the SRPM
before committing those changes. 🤦🏻‍♂️

As for the tests, the azure tests require two bits of helper code from the git
repository itself that are not installable on their own and they are not
published to pypi. In addition, the majority of the tests need network access
to download certain things and talk to Azure's API. That's why I haven't found
a way to run tests here yet.

If I try your testing method, I get:

  ImportError: Start directory is not importable: './tests/'

pytest *does* work, but then I run into the network access problems and the
missing helpers from the upstream repository. The whole upstream repository is
about ~ 150-160MB and each of these little components are released
independently from that repository.


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
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/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux