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

Update Simple Library Server #1952

Merged
merged 13 commits into from
Aug 18, 2021
Merged

Update Simple Library Server #1952

merged 13 commits into from
Aug 18, 2021

Conversation

radeusgd
Copy link
Member

@radeusgd radeusgd commented Aug 13, 2021

Pull Request Description

Updates the simple library server in preparation for our deployment.

  • Support for the PORT environment variable.
  • Making it more robust: atomic move for committing the upload, handling signals for graceful shutdown.

Important Notes

Checklist

Please include the following checklist in your PR:

  • The documentation has been updated if necessary.
  • All code conforms to the Scala, Java, and Rust style guides.
  • All documentation and configuration conforms to the markdown and YAML style guides.
  • All code has been tested where possible.

@radeusgd radeusgd marked this pull request as ready for review August 16, 2021 12:20
@radeusgd radeusgd self-assigned this Aug 16, 2021
@radeusgd radeusgd added Category: Distribution -tooling Category: tooling p-high Should be completed in the next sprint labels Aug 16, 2021
@radeusgd radeusgd force-pushed the wip/rw/update-library-server branch from 2f4f4c8 to 91b28b5 Compare August 16, 2021 15:29
@radeusgd radeusgd force-pushed the wip/rw/update-library-server branch from 91b28b5 to ff11f62 Compare August 17, 2021 11:11
@radeusgd radeusgd merged commit 0a60e51 into main Aug 18, 2021
@radeusgd radeusgd deleted the wip/rw/update-library-server branch August 18, 2021 08:01
iamrecursion pushed a commit that referenced this pull request Aug 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-tooling Category: tooling p-high Should be completed in the next sprint
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants