Skip to content

DVC-8773 [feat]: add overrides list command - #283

Merged
kaushalkapasi merged 1 commit into
mainfrom
DVC-8775-overrides-list
Sep 11, 2023
Merged

DVC-8773 [feat]: add overrides list command#283
kaushalkapasi merged 1 commit into
mainfrom
DVC-8775-overrides-list

Conversation

@kaushalkapasi

Copy link
Copy Markdown
Contributor
  • feat: adds overrides list command
  • chore: fix output for overrides get table output
  • chore: fix output for tables to include newline after for better readability

@kaushalkapasiGraphite App

Copy link
Copy Markdown
ContributorAuthor

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@kaushalkapasi
kaushalkapasi requested a review from a teamSeptember 11, 2023 15:42
Comment threadsrc/commands/overrides/get.ts
description: 'The key or id of the Environment to get Overrides for',
}),
...Base.flags,
...ux.table.flags(),

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.

any reason for removing these?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

unused flags for this command :)

@kaushalkapasi
kaushalkapasi merged commit bd13043 into mainSep 11, 2023
@kaushalkapasi
kaushalkapasi deleted the DVC-8775-overrides-list branch September 11, 2023 15:56
jonathannorris added a commit that referenced this pull request Aug 5, 2026
* chore: resolve open dependabot security alerts
- brace-expansion ^1.1.16 -> ^1.1.18 (high, alerts #285/#286)
- brace-expansion ^2.1.2 -> ^2.1.4 (high, alerts #282/#287)
- undici ^7.28.0 -> ^7.29.0 (medium/high, alerts #289-#293)
- ip-address ^10.1.1 -> ^10.3.1 (medium/high, alerts #283/#284/#294)
- hono ^4.12.25 -> ^4.12.34 (medium, alert #295)
- fast-uri ^3.1.4 -> ^3.1.5 (high, alert #288)
- wrangler ~4.116.0 (via miniflare 4.20260730.0 -> sharp 0.35.2, high, alert #276)
* chore: exclude @img/sharp-libvips-linux-x64@1.3.1 from license check
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

@kaushalkapasi@suthar26