Prevent 1Mb limit#48
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #48 +/- ##
=======================================
Coverage 98.51% 98.51%
=======================================
Files 4 4
Lines 538 538
Branches 51 51
=======================================
Hits 530 530
Misses 4 4
Partials 4 4 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Apologies for working on the same repo at the same time, but I thought I had better alert you to this recent change: #47 so that you can avoid re-using the same version number. If you want to bump the version number, please bump up to at least 0.1.9. |
|
No probs - I had forgotten to bump the version at all! This PR was mainly so I wouldn't forget... |
|
To test out the change I'd like to try uploading something, but I'm not sure where that feature exists in packit... |
@david-mears-2 This feature is actually used via orderly, to push a packet, having first added a location These functions use packit endpoints to authenticate and add new packets from the user's local orderly instance. @weshinsley Is this deployed somewhere that we can test against? |
EmmaLRussell
left a comment
There was a problem hiding this comment.
I did the following:
Create location for reside instance on packit-dev
orderly_location_add_packit("packit-dev-reside", "https://reside-dev.packit-dev.dide.ic.ac.uk")
Push a packet I had already created
orderly_location_push("20251212-103140-c210108b", location="packit-dev-reside")
Error:
Error: Payload Too Large
I then installed this branch of packit-deploy on packit-dev, redeployed packit and a reattempt to push the packet worked this time.
The packit-deploy branch is still on packit-dev, we should reset once this branch is merged and released.
No description provided.