-
Notifications
You must be signed in to change notification settings - Fork 104
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
December 2024 fixes #1088
December 2024 fixes #1088
Conversation
Gonna fix some councils just like the good old days @robbrad I appreciate you're very busy this time of year, but I noticed the Allure report has a 404 too (though I'm not raising it as an issue as its not directly plugin related lol). |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1088 +/- ##
=======================================
Coverage 71.93% 71.93%
=======================================
Files 9 9
Lines 1037 1037
=======================================
Hits 746 746
Misses 291 291 ☔ View full report in Codecov by Sentry. |
Been trying to fix it today but no luck yet. I might separate out the publish to a different job so I can rerun it. At the moment it's an hour to find out if my changes worked 🥹 |
fix: robbrad#1080 (cherry picked from commit 6f580b3)
fix: robbrad#1076 (cherry picked from commit 1eab20c)
(cherry picked from commit 8e70e6a)
Hope you don't mind @m26dvd, I cherry-picked some of your fixes to go in here 😄 |
No problem! I’ve not had time recently to do any more and it makes sense to batch release them rather than raise a PR for one fix! |
@dp247 I’ve also got a new scraper for 1078 if you fancy including it here too! |
Sure thing, I'll pick it after work 😊 |
fix: robbrad#1078 (cherry picked from commit 89d9366)
…orwich City Council
…nual mapping for Rugby Borough Council
@dp247 : are these two failures ok ? https://github.com/robbrad/UKBinCollectionData/actions/runs/12403956484/job/34628376787?pr=1088#step:7:567 If you are happy I will approve |
I'll take a look at them, though they don't seem to be breaking anything other than tests. Gonna stick another couple of issues in here too |
@dp247 : can I ask a small favour can you make a small change to one of the council file to invoke the latest work flow? I want to see the int tests report work for partial The change doesnt have to be anything that would affect your work |
Haha, I can do... tomorrow when I get home 😅 |
Don't bother. For some reason unknown it's still not working |
fix: robbrad#1080 (cherry picked from commit 6f580b3)
Bug fixes
Closes Unexpected Error #1063
Closes MertonCouncil: Unexpected error: No connection adapters were found for 'python collect_data.py MertonCouncil #1072
Closes Missing ‘type’ or ‘collectiondate’ error in logs #1073
Closes Error reported after update to 0.122 #1076
Closes Swale Updated URL #1080
Closes Integration no longer works/sets up since recent update #1084
Closes Food Waste Bin Date Incorrect & request to add bin colours (if possible) #1087
Closes Bromley Borough Council dates are shifted by one year #1089
Closes Walsall Council No Longer Working #1101
Add missing backticks to README.md
Test fixes
Additions