This is an intermediate merge to get a review on what I've done so far and have a beta branch (v3).

Features:

Rename modules to match the ETL process:
- load.py becomes extract.py
- updaters.py becomes the new load.py
Remove transform logic from agol updating code (in load.py)- all pre-upload dataframe merging and prep will be in transform.py or client code
Rewrite all agol update logic in load.py
- Uses class functions to create objects so the client doesn't have to do that step
- Splits updates out into four distinct operations: add features, remove features, update features, truncate and load features
- All adds/updates use a single method that calls .append (closes #14 )
- Consolidated field checks, added a bunch of new ones (ref #26 )
- FeatureServiceOverwriter becomes the truncate_and_load_features method of FeatureServiceUpdater
Clients now import all of palletjack and call classes as palletjack.module.class, with module names following the ETL process. Old version added each class to __init__.py and required users to reference classes directly. This makes it more clear what step in the process a class belongs to.

Codecov Report

Base: 91.99% // Head: 89.67% // Decreases project coverage by -2.31% :warning:

Coverage data is based on head (f267be7) compared to base (f9a0e79). Patch coverage: 90.75% of modified lines in pull request are covered.

:exclamation: Current head f267be7 differs from pull request most recent head e76106e. Consider uploading reports for the commit e76106e to get more accurate results

Additional details and impacted files

```diff @@ Coverage Diff @@ ## v3 #31 +/- ## ========================================== - Coverage 91.99% 89.67% -2.32% ========================================== Files 6 6 Lines 662 833 +171 Branches 93 117 +24 ========================================== + Hits 609 747 +138 - Misses 49 77 +28 - Partials 4 9 +5 ``` | [Impacted Files](https://codecov.io/gh/agrc/palletjack/pull/31?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=agrc) | Coverage Δ | | |---|---|---| | [src/palletjack/extract.py](https://codecov.io/gh/agrc/palletjack/pull/31?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=agrc#diff-c3JjL3BhbGxldGphY2svZXh0cmFjdC5weQ==) | `88.88% <73.68%> (ø)` | | | [src/palletjack/load.py](https://codecov.io/gh/agrc/palletjack/pull/31?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=agrc#diff-c3JjL3BhbGxldGphY2svbG9hZC5weQ==) | `85.51% <85.51%> (ø)` | | | [src/palletjack/transform.py](https://codecov.io/gh/agrc/palletjack/pull/31?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=agrc#diff-c3JjL3BhbGxldGphY2svdHJhbnNmb3JtLnB5) | `96.77% <92.30%> (-3.23%)` | :arrow_down: | | [src/palletjack/utils.py](https://codecov.io/gh/agrc/palletjack/pull/31?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=agrc#diff-c3JjL3BhbGxldGphY2svdXRpbHMucHk=) | `94.03% <99.50%> (-1.21%)` | :arrow_down: | | [src/palletjack/\_\_init\_\_.py](https://codecov.io/gh/agrc/palletjack/pull/31?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=agrc#diff-c3JjL3BhbGxldGphY2svX19pbml0X18ucHk=) | `100.00% <100.00%> (ø)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=agrc). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=agrc)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

agrc / palletjack

Intermediate Merge: Initial refactor, AGOL updates #31

Codecov Report