This changes some of the keyword arguments so that they are a little more consistently named in the API and avoid renaming the builtin function id. Kind of a small change, but I wanted to get it in before v3. Basically anything that was passed as id has a more descriptive argument now: id in the context of a Status object changes to status_id, message_id for DirectMessages, and woeid for the Where On Earth ids. Similarly, for DestroyBlock, it's now user_id rather than just id.
Documentation is included for everything here. All previous tests still pass, but some of the behavior is breaking since kwargs have been renamed.
I also noticed that make clean kept deleting my vim undo files, so I hope no one minds that those are excluded from the target.
"status_id" is more common within the api, but there are a couple methods that use "id" to spec the same thing, which is confusing and inconsistent.
"woeid" avoids shadowing a builtin, but is also more expressive.
This changes some of the keyword arguments so that they are a little more consistently named in the API and avoid renaming the builtin function
id. Kind of a small change, but I wanted to get it in before v3. Basically anything that was passed asidhas a more descriptive argument now:idin the context of a Status object changes tostatus_id,message_idfor DirectMessages, andwoeidfor the Where On Earth ids. Similarly, for DestroyBlock, it's nowuser_idrather than justid.Documentation is included for everything here. All previous tests still pass, but some of the behavior is breaking since kwargs have been renamed.
I also noticed that
make cleankept deleting my vim undo files, so I hope no one minds that those are excluded from the target.Sound good, @bear?
This change is