Clean up PropertyLoadSaver interface.

This removes GetMetaDefault and Problem from the interfaces, and makes GetPLS
errors into panics, on the assumption that if you pass it a bad datatype, you
can't really do anything about it without recompiling, and failing fast is better.

This also fixes a bug where doing any operations on an invalid (non datastore-
compatible) struct would return the unhelpful "unable to extract $kind" error.

R=dnj@chromium.org, hinoka@chromium.org, martiniss@chromium.org, vadimsh@chromium.org
BUG=

Review URL: https://codereview.chromium.org/1516173002
15 files changed
tree: d6aa97b029bb418835f621b3affe649cb696f603
  1. filter/
  2. impl/
  3. infra/
  4. service/
  5. .travis.yml
  6. codereview.settings
  7. doc.go
  8. LICENSE
  9. pre-commit-go.yml
  10. PRESUBMIT.py
  11. README.md
README.md

gae: A Google AppEngine SDK wrapper

designed for testing and extensibility

THIS PACKAGE HAS NO API COMPATIBILITY GUARANTEES. USE UNPINNED AT YOUR OWN PERIL.

(but generally it should be pretty stableish).

GoDoc Build Status Coverage Status

Installing

go get -u github.com/luci/gae/...

Why/What/How

See the godocs.

Versioning

  • Branch master contains the latest code.

Contributing

  • Sign the Google CLA.
  • Make sure your user.email and user.name are configured in git config.
  • Install the pcg git hook: go get -u github.com/maruel/pre-commit-go/cmd/... && pcg

Run the following to setup the code review tool and create your first review:

git clone https://chromium.googlesource.com/chromium/tools/depot_tools.git $HOME/src/depot_tools
export PATH="$PATH:$HOME/src/depot_tools"
cd $GOROOT/github.com/luci/gae
git new-branch work
# or `git checkout -b work origin/master` if you like typing more.

# hack hack

git commit -a -m "This is awesome\nR=joe@example.com"
# This will ask for your Google Account credentials.
git cl upload -s
# Wait for LGTM over email.
# Check Commit Queue checkbox in Rietveld codereview site.
# See it tested and landed automatically.

Use git cl help and git cl help <cmd> for more details.