-
Notifications
You must be signed in to change notification settings - Fork 406
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
feat: Add logout
command
#3605
feat: Add logout
command
#3605
Conversation
for more information, see https://pre-commit.ci
…argilla into feature/logout-command
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## develop #3605 +/- ##
===========================================
- Coverage 90.53% 90.49% -0.05%
===========================================
Files 261 264 +3
Lines 13768 13807 +39
===========================================
+ Hits 12465 12494 +29
- Misses 1303 1313 +10
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eshwarhs Thansk for your pull request! I wrote you some comments that needs to be addressed before we can merge this branch.
Awesome @eshwarhs, this is really useful :) |
Description
This PR adds the
logout
command that will allow to logout from Argilla server and remove the locally stored credentials.Closes #3585
Type of change
How Has This Been Tested
Tested manually in a local development environment and added tests to cover the additions and changes.
Checklist