FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

small changes test and added sleep time between search func by munkhuushmgl · Pull Request #2547 · GoogleCloudPlatform/java-docs-samples · GitHub

Repository navigation

small changes test and added sleep time between search func - #2547

Merged
lesv merged 6 commits into
masterfrom
small-bugs-jobs-v3
Apr 1, 2020
Merged

lesv merged 6 commits into
masterfrom
small-bugs-jobs-v3

Conversation

Copy link
Copy Markdown
Contributor

Fixes #2261 #2232

munkhuushmgl requested a review from a team April 1, 2020 21:00
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 1, 2020
Comment on lines +34 to +36
* <p>- Construct a featured job
*
* - Search featured job
* <p>- Search featured job

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Doc samples don't need <p>

// requisition id should be a unique Id in your system.
String requisitionId =
"featuredJob:" + String.valueOf(new Random().nextLong());
String requisitionId = "featuredJob:" + String.valueOf(new Random().nextLong());

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Ok, but it's better to use a real UUID than a Random as that's not guaranteed to truly be random.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I will make these changes

lesv merged commit 6a24be1 into master Apr 1, 2020
lesv deleted the small-bugs-jobs-v3 branch April 1, 2020 22:07
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

SampleTests: customAttributeSampleTest failed

4 participants


Back | FazBrowse Home | New Git URL