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

Fix for enhancement#1001: Google Code-in Task to update the error message #189

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Rithi-21
Copy link

@Rithi-21 Rithi-21 commented Dec 8, 2019

Fix for enhancement#1001: Google Code-in Task to update the error message

As per Mentor's feedback, I fixed the error message so that it only show the host details of the host url.

======

This is for Google Code-in Task. Updated the error message to add the URL of the API end point to the end of the existing error message. Currently it shows as " Please check the Host URL" and it is modified to add the host url at the end of this message. ex: if the CLI is not able to connect to https://api.evalai.example the error should be "Could not establish connection to EvalAI. Please check the Host URL: https://api.evalai.example".

…sage

Fix for enhancement#1000: Google Code-in Task to update the error message
This is for Google Code-in Task. Updated the error message to add the URL of the API end point to the end of the existing error message.  Currently it shows as " Please check the Host URL" and it is modified to add the host url at the end of this message. ex: if the CLI is not able to connect to https://api.evalai.example the error should be "Could not establish connection to EvalAI. Please check the Host URL: https://api.evalai.example".
…sage

Fix for enhancement#1001: Google Code-in Task to update the error message
This is for Google Code-in Task. Updated the error message to add the URL of the API end point to the end of the existing error message.  Currently it shows as " Please check the Host URL" and it is modified to add the host url at the end of this message. ex: if the CLI is not able to connect to https://api.evalai.example the error should be "Could not establish connection to EvalAI. Please check the Host URL: https://api.evalai.example".

As per the Mentor feedback, I fixed the host url to display only the host details.
Copy link
Member

@Ram81 Ram81 left a comment

Choose a reason for hiding this comment

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

@Rithi-21 LGTM 👍

@yashdusing
Copy link

LGTM. nvm previous comment. 👍

Rithi-21 and others added 6 commits December 10, 2019 17:49
Fix for enhancement#1002: Google Code-in Task to add a command

Added a command for displaying the text from stdout file in terminal. Assuming the stdout file is located under ~/.evalai/stdout.txt. In order to  test this functionality, create a sample stdout.txt file under ~/.evalai/. and issue the following command 'evalai display' will output the stdout.txt file's content.
incorrect task.
…R DOWNLOAD_FILE CLI COMMAND

Fix for enhancement#1003: Google Code-in Task to ADD DOCUMENTATION FOR DOWNLOAD_FILE CLI COMMAND

This is for Google Code-in Task. As per Mentor's feedback, I updated the docs/readme.md file to provide the steps how to to use the cli command for download_file . Please review the PR and merge it to the master branch
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.

3 participants