Skip to content
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

Use a separate directory to store registration information #582

Merged
merged 3 commits into from
Nov 10, 2023

Conversation

sundargates
Copy link
Contributor

Context

^^^

Checklist

  • ./gradlew build compiles code correctly
  • Added new tests where applicable
  • ./gradlew test passes all tests
  • Extended README or added javadocs where applicable

Comment on lines +255 to +259

@Override
public File getRegistrationStoreDir() {
return null;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This will not and should not be used by the worker itself. Hence, it returns a null for the time being.

Copy link

github-actions bot commented Nov 10, 2023

Test Results

546 tests  ±0   538 ✔️ ±0   6m 59s ⏱️ -8s
129 suites ±0       8 💤 ±0 
129 files   ±0       0 ±0 

Results for commit 72b8a6b. ± Comparison against base commit 8f5ad73.

♻️ This comment has been updated with latest results.

@sundargates sundargates had a problem deploying to Integrate Pull Request November 10, 2023 00:38 — with GitHub Actions Failure
@sundargates sundargates merged commit e5fad93 into Netflix:master Nov 10, 2023
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant