Skip to content

Conversation

@juanangp
Copy link
Member

@juanangp juanangp commented May 25, 2022

juanangp Ok: 11

Potential fix for #169

  • Now REST_PATH is not set in case of testing are enabled. I checked that is working fine with the test currently implemented, however we might have potential issues in case REST_PATH is required. However, REST_PATH cannot be set during test since installation path is not present at this stage...
  • Doesn't change the logic when test are disabled.
  • Added newRESTError and RESTWarning functions instead of std::cout

@juanangp juanangp requested review from jgalan, lobis and nkx111 May 25, 2022 11:10
@juanangp juanangp merged commit dd71405 into master May 25, 2022
@juanangp juanangp deleted the issue169 branch May 25, 2022 16:29
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.

4 participants