SNEWS2 / SNEWS_Publishing_Tools

Publishing Tool for SNEWS
BSD 3-Clause "New" or "Revised" License
2 stars 2 forks source link

Update dependency version #96

Closed justinvasel closed 4 months ago

justinvasel commented 6 months ago

PR to accompany Issue #91

Resolves: #91, #78

justinvasel commented 6 months ago

Following the merger of this PR, I recommend tagging a new release, so that snews_cs can pull in the latest version and upgrade its own dependencies.

justinvasel commented 6 months ago

Apologies for the email traffic... I had to do a bunch of tests to get the github workflow working with the new versions of hop-client and pytest. I ultimately ended up squashing all those commits and resubmitting the PR to keep the commit history clean. Probably can't say the same for your inbox though... :/

KaraMelih commented 6 months ago

I think I was able to follow the flow. I can try to test this branch on my local machine next Monday

KaraMelih commented 6 months ago

Hi, I could only get back to these PRs now. I checked the existing PRs addressing the SQL fixes and datetime conversions. Now, these branches https://github.com/SNEWS2/SNEWS_Publishing_Tools/pull/90 and https://github.com/SNEWS2/SNEWS_Coincidence_System/pull/83 should play nicely with each other. While they should in principle not affect this PR, in order for me to run updated dependencies I need working versions of the scripts.