petl-developers / petl

Python Extract Transform and Load Tables of Data
MIT License
1.24k stars 193 forks source link

Document how to use convert with custom function #544

Closed javidy closed 3 years ago

javidy commented 3 years ago

Fixes #542

This PR has the objective of...giving an example of using a custom function with petl.transform.conversions.convert()

Changes

  1. Improved the docs about...petl.transform.conversions.convert() providing example usage of custom function

Checklist

Checklist for for pull requests including new code and/or changes to existing code...

coveralls commented 3 years ago

Coverage Status

Coverage increased (+0.09%) to 90.007% when pulling 66f0716be94ae787fb30c8c3da95135e58c54fbf on javidy:patch-1 into e9b3e87f921f8310f058de905953c3cdd65c2b57 on petl-developers:master.

javidy commented 3 years ago

hi @juarezr , would you be kind to review my PR :)

javidy commented 3 years ago

thanks a lot @juarezr , it was my first PR and I'm happy that it was merged :) hope I'll have many more valuable contributions in future.

juarezr commented 3 years ago

Welcome @javidy and @carlcorder !!!

:1st_place_medal:

juarezr commented 3 years ago

It looks like docs building / deploying CI is broken now...

javidy commented 3 years ago

@juarezr how can I help to fix the issue? Would be happy to do that.

javidy commented 3 years ago

I'm running doctests using tox as mentioned in documentation. Maybe it'll give me a clue.

juarezr commented 3 years ago

Local doctests and CI are working fine.

The problem could be someplace involving: