Skip to content

Conversation

@chrisrossi
Copy link
Contributor

Adds back some missing documentation to CONTRIBUTING.rst that covers
creating indexes and populating test data for system tests.

Also increases the number of timestamp_key entities created by
populate_datastore.py to 10000, so that system tests can pass.

Adds back some missing documentation to `CONTRIBUTING.rst` that covers creating indexes and populating test data for system tests. Also increases the number of `timestamp_key` entities created by `populate_datastore.py` to 10000, so that system tests can pass.
@chrisrossi chrisrossi requested review from crwilcox and tseaver August 11, 2020 19:54
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 11, 2020
@chrisrossi
Copy link
Contributor Author

If there's a more elegant way to do this, I'm open to suggestions.

I didn't check in changes to synth.metadata after running synthtool. I'm not entirely sure on the policy for that.

@tseaver
Copy link
Contributor

tseaver commented Aug 11, 2020

@chrisrossi If you don't check them in, then the bot will submit a PR tomorrow. No biggie. Usually, though, I check in synth.metadata if synthtool produced any other changes (like the ones in CONTRIBUTING.md).

@chrisrossi chrisrossi merged commit 8bb17ea into googleapis:master Aug 12, 2020
@chrisrossi chrisrossi deleted the fix-system-tests branch August 12, 2020 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

2 participants