Print statement outside loop #306
Conversation
Small fix to finally put print statement outside main loop! Also, I have been using desktop version of this script for quite a while now as there's something up with the ifiscripts version being pulled down. Going to finally sort this out too.
|
Hey, thanks for this. If there's a issue with updating the as11fixity file via update.py, then it's because the version of as11fixity.py within ifigit/ifiscripts has been edited at some point since the last Also it looks like you're working from a very old version of as11fixity, as I rewrote it to be more PEP-08 compliant a while back. I think the functionality was identical, it was just the style that was made more in line with ifiscripts. Which ingest station was having the issue? |
|
Both of them I think but different issues I think. This is a old issue that I really should have gotten around to before. If I delete the as11fixity.py file from both ifigit folders and then do a new git pull, should this fix the issue? |
|
Yes, that'll fix it, then you can see if there's still an issue with that print statement outside the loop and you can send an updated Pull Request. Cheers Eoin. |
|
Hey, checking in on this and if this PR is still relevant after sorting out the outdated script issue? |
Small fix to finally put print statement outside main loop! Also, I have been using desktop version of this script for quite a while now as there's something up with the ifiscripts version being pulled down. Going to finally sort this out too.