Skip to content

feat: add command to display current project - #297

Merged
nsalamad merged 1 commit into
mainfrom
DVC-8167-display-current-project
Sep 26, 2023
Merged

feat: add command to display current project#297
nsalamad merged 1 commit into
mainfrom
DVC-8167-display-current-project

Conversation

@nsalamad

Copy link
Copy Markdown
Contributor

projects current
image

status
image

projects list
image

keys get
image

login again
image

@nsalamad
nsalamad requested a review from a teamSeptember 26, 2023 14:21
@nsalamad
nsalamadforce-pushed the DVC-8167-display-current-project branch 2 times, most recently from 38d64a4 to c2097bcCompareSeptember 26, 2023 14:22
Comment threadsrc/commands/keys/get.ts Outdated
Comment threadsrc/commands/status/index.ts Outdated
@laurawarr

Copy link
Copy Markdown
Contributor

Nice work! I like that you included the project on all these commands rather than just the status 👌

@nsalamad
nsalamadforce-pushed the DVC-8167-display-current-project branch 3 times, most recently from 2f8add8 to 89a5c33CompareSeptember 26, 2023 16:30
Comment threadsrc/commands/base.ts Outdated

@laurawarrlaurawarr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

added one comment, otherwise looks good 👍

@nsalamad
nsalamadforce-pushed the DVC-8167-display-current-project branch from 89a5c33 to 3c6b5c8CompareSeptember 26, 2023 19:52
@nsalamad
nsalamad merged commit 66f5155 into mainSep 26, 2023
@nsalamad
nsalamad deleted the DVC-8167-display-current-project branch September 26, 2023 20:52
jonathannorris added a commit that referenced this pull request Aug 12, 2026
- js-yaml 4.3.0 -> 4.3.1 (high, alert #298)
- js-yaml 3.15.0 -> 3.15.1 (high, alert #297)
- nanoid 5.1.6 -> 5.1.16 (high, alert #296)
- nanoid 3.3.16 -> 3.3.18 (high, alert #299)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@nsalamad@laurawarr