crs4 / hl7apy

Python library to parse, create and handle HL7 v2 messages.
http://crs4.github.io/hl7apy/
MIT License
215 stars 85 forks source link

Pushing a new release to Pypi to remove extra debug statements in stdout? #106

Closed ethan1uphealth closed 4 months ago

ethan1uphealth commented 1 year ago

Hi,

I noticed when using the hlyapy v1.3.4 package from Pypi there are still some debug statements being output to stdout. It looks like this is coming from a print() statement in factories:102. In the develop branch, it looks like this extra print statement was removed in an earlier PR.

Is there a way someone could update the Pypi package to make it easier to get rid of those debug statements?

Thanks!

Example:

(000)000-0000 is not an HL7 valid NM value

svituz commented 1 year ago

Hi @ethan1uphealth, we need to schedule a new release with some bug fixes and we'll take care of printing too.

yankov-sdx commented 5 months ago

Is there any chance a new release is going to come out with this print statement removed soon? I see potential for PHI to get printed out that should not appear in the logs. Planning on using a github branch as a work around in the mean time, but would be nice to be able to go pypi instead.

svituz commented 4 months ago

Hi I released version 1.3.5