Skip to content

Rebrand Cloud Job Discovery to Cloud Talent Solution and add the V3 code samples #1166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

xinyunh0929
Copy link
Contributor

Move the previous samples from cjd_sample/ to v2/ and have a new folder v3/ holding v3 samples.
Please hold the test until our V3 API is public in 8/3/2018.

Copy link
Member

@dzlier-gcp dzlier-gcp left a comment

Choose a reason for hiding this comment

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

Taking a look at this PR, but note that presubmit and jobs tests have failed. Looks like the api isn't properly enabled in the java-docs-samples-testing project, but please look through and make sure.

https://source.cloud.google.com/results/invocations/6de5dff1-b0d4-4c77-b3b0-12d84ded6d47/targets/github%2Fjava-docs-samples%2Fjobs%2Fv3%2Ftarget%2Fsurefire-reports/tests

@xinyunh0929
Copy link
Contributor Author

Sorry, forgot to unify the setup steps.

dzlier-gcp
dzlier-gcp previously approved these changes Aug 30, 2018
new CreateJobRequest().setJob(jobToBeCreated);

Job jobCreated = talentSolutionClient.projects().jobs().create(DEFAULT_PROJECT_ID, createJobRequest).execute();
//Job jobCreated = ta.jobs().create(createJobRequest).execute();
Copy link
Member

Choose a reason for hiding this comment

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

delete commented out line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

.setSessionId("HashedSessionID")
// Domain of the website where the search is conducted
.setDomain(
"www.google.com");
Copy link
Member

Choose a reason for hiding this comment

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

newline not necessary

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

bout.reset();
}

// @Test
Copy link
Member

Choose a reason for hiding this comment

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

Remove commented out test.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Add them back.

@dzlier-gcp
Copy link
Member

Looks like checkstyle errors are causing Kokoro to still fail. You can run this with the maven target checkstyle:checkstyle

@dzlier-gcp dzlier-gcp closed this Sep 4, 2018
@dzlier-gcp dzlier-gcp reopened this Sep 4, 2018
@xinyunh0929
Copy link
Contributor Author

Done.

@dzlier-gcp
Copy link
Member

@dpebot Merge when green

@dpebot
Copy link
Contributor

dpebot commented Sep 4, 2018

Okay! I'll merge when all statuses are green and all reviewers approve.

@dpebot dpebot self-assigned this Sep 4, 2018
@xinyunh0929
Copy link
Contributor Author

Thank you very much!

@xinyunh0929
Copy link
Contributor Author

One question, why it is still "Kokoro CI Expected — Waiting for status to be reported"

@dzlier-gcp
Copy link
Member

Ah, sorry I need to trigger a test run manually. Starting one now.

@dzlier-gcp dzlier-gcp added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 5, 2018
@kokoro-team kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 5, 2018
@dzlier-gcp dzlier-gcp merged commit d72fd89 into GoogleCloudPlatform:master Sep 5, 2018
@xinyunh0929
Copy link
Contributor Author

xinyunh0929 commented Sep 5, 2018 via email

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.

5 participants