Comments (3)
Hi Ricardo, thanks for catching this.
As a general rule, I'd like to try keep any breaking changes as small as possible.
In this case, what if:
- We make
update-map
optional (a breaking change) but - Retain support for
update-map
with the (corrected) warning
This way folks adopting to the break will be forced to update only the arg format, not the update logic.
Thanks again for all your work on Faraday btw, looking forward to seeing the results :-)
from faraday.
This way folks adopting to the break will be forced to update only the arg format, not the update logic.
Sounds good, I'll move it into the map with the rest of the options and update the comments.
Thanks again for all your work on Faraday btw, looking forward to seeing the results :-)
My pleasure. I'll soon need to get back to the actual project, so I'll start making changes as I go instead of in a big push, but it was a good way to get acquainted with the library.
I'll wrap this issue, then open up a request for that alpha release you mentioned. I'll probably write a brief tutorial now that's it's all fresh in my mind, and it would be useful to have an official alpha/beta release out.
Cheers!
from faraday.
Done.
from faraday.
Related Issues (20)
- Remove support for `:expected` HOT 3
- attribute_not_exists not respected consistently HOT 3
- Support for Streams HOT 8
- missing ScanIndexForward option with far/scan HOT 1
- Request: custom keywordization routine HOT 3
- Is the scan function safe for large tables? HOT 7
- Upgrade to 1.11.16+ of aws-sdk-java for task-level IAM role support in AWS ECS HOT 9
- Add support for DynamoDB Transactions HOT 6
- Upgrade to 1.11.459 of aws-sdk-java for management of On Demand tables
- Upload all doc versions to http://taoensso.github.io/faraday/{version}
- Porting to AWS SDK v2 HOT 16
- missing page-size/max-result-size option with query HOT 1
- Support empty strings HOT 4
- not listing tables HOT 14
- Setting `region` in client-opts causes ClassCastException
- String index names cause type error HOT 3
- Local docker example from readme only with `-p 8000:8000` arg HOT 1
- Possibly confusing dependencies found HOT 3
- FYI: have pushed some superficial changes HOT 1
- Vulnerability CVE-2024-21634 HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from faraday.