Fail `gix tag list` when JSON is requested

Christoph Rüßler committed 1 year ago

Commit fb2766b1 parent fa1287f3 files · +9 −3Browse files

Unified · Split

3 files

 use gix::bstr::{BStr, BString, ByteSlice};
+use crate::OutputFormat;
 #[derive(Eq, PartialEq, PartialOrd, Ord)]
 enum VersionPart {
     String(BString),
     }
 }
-pub fn list(repo: gix::Repository, out: &mut dyn std::io::Write) -> anyhow::Result<()> {
+pub fn list(repo: gix::Repository, out: &mut dyn std::io::Write, format: OutputFormat) -> anyhow::Result<()> {
+    if format != OutputFormat::Human {
+        anyhow::bail!("JSON output isn't supported");
+    }
     let platform = repo.references()?;
     let mut tags: Vec<_> = platform

@@ -3,7 +3,7 @@ use std::path::Path; · 2 unmodified lines
3		3
4	pub use gix_discover::*;	4	pub use gix_discover::*;
5		5
6	use crate::{bstr::BString, Repository, ThreadSafeRepository};	6	use crate::{bstr::BString, ThreadSafeRepository};
7		7
8	/// The error returned by [`crate::discover()`].	8	/// The error returned by [`crate::discover()`].
9	#[derive(Debug, thiserror::Error)]	9	#[derive(Debug, thiserror::Error)]

@@ -1312,7 +1312,7 @@ pub fn main() -> Result<()> { · 1311 unmodified lines
1312	progress,	1312	progress,
1313	progress_keep_open,	1313	progress_keep_open,
1314	None,	1314	None,
1315	move \|_progress, out, _err\| core::repository::tag::list(repository(Mode::Lenient)?, out),	1315	move \|_progress, out, _err\| core::repository::tag::list(repository(Mode::Lenient)?, out, format),
1316	),	1316	),
1317	},	1317	},
1318	Subcommands::Tree(cmd) => match cmd {	1318	Subcommands::Tree(cmd) => match cmd {

Keyboard shortcuts