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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants