Skip to content

Cleaned up server scripts#1

Open
andersevenrud wants to merge 1 commit intoBurnaSmartLab:masterfrom
andersevenrud:refactor/minor-cleanup
Open

Cleaned up server scripts#1
andersevenrud wants to merge 1 commit intoBurnaSmartLab:masterfrom
andersevenrud:refactor/minor-cleanup

Conversation

@andersevenrud
Copy link
Copy Markdown
Contributor

@andersevenrud andersevenrud commented Jul 14, 2021

This commit cleans up some of the server-side code.

  • Use async/await exclusively
  • Added error handling wrapper for routes
  • Added VFS call wrapper for handling calls with sessions
  • Improved discovery http call error handling
  • Use streams whenever possible
  • Removed some defunct code
  • Various cleanups

@andersevenrud andersevenrud force-pushed the refactor/minor-cleanup branch 2 times, most recently from 3c3c0ea to 2789cc0 Compare July 15, 2021 00:08
@andersevenrud andersevenrud force-pushed the refactor/minor-cleanup branch from 2789cc0 to a58b2bb Compare July 15, 2021 00:20
@mahsashadi
Copy link
Copy Markdown
Collaborator

Hi. Thanks for your refactoring.
Unfortunately git watch was not activated for me, so I didn't notified this long time. Sorry about that.

Would you please rebase your branch? @andersevenrud

@andersevenrud
Copy link
Copy Markdown
Contributor Author

andersevenrud commented Jan 25, 2023 via email

@mahsashadi
Copy link
Copy Markdown
Collaborator

You are right. We had long delay, sorry about that.
We have some commits during this time, so It will take your time.
Thank you in advance.

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.

2 participants