Skip to content

Ux/conan output generate - #20085

Merged
czoido merged 20 commits into
conan-io:develop2from
memsharded:ux/conan_output_generate
Jun 22, 2026
Merged

czoido merged 20 commits into
conan-io:develop2from
memsharded:ux/conan_output_generate

Conversation

@memsharded

@memsharded memsharded commented Jun 16, 2026

Copy link
Copy Markdown
Member

Changelog: Fix: Improve output messages for the generate() step.
Docs: Omit

For #19810

This is the resulting output:
image

I am not even sure that tabs are the best, or if we needed more headers and spacing instead.
Same for already merged:

image

@memsharded
memsharded requested a review from czoido June 16, 2026 11:16
@perseoGI

Copy link
Copy Markdown
Contributor

I'm testing this branch locally with an example of a very long recipe name (issue related one) and these are the differences:

develop2 branch:
image

PR changes:
image

I'm not seeing the recipe name title on the generate step! Is that expected?

@memsharded
memsharded marked this pull request as ready for review June 19, 2026 11:16
@memsharded

Copy link
Copy Markdown
Member Author

I am finally going with a >>> chevron (no indents) Example:

image

The inner pkg/version scope for source, build, package steps hasn't been removed yet, too many tests already changed, that will follow later in other PRs.

@perseoGI

Copy link
Copy Markdown
Contributor

I am finally going with a >>> chevron (no indents) Example:

I like it!

The inner pkg/version scope for source, build, package steps hasn't been removed yet, too many tests already changed, that will follow later in other PRs.

But the how is going to be displayed the pkg/version?

Is it going to be:

Generating files for <pkg/version>
<logs>

Building <pkg/version>
<logs>

?

@memsharded

Copy link
Copy Markdown
Member Author

Generating files for <pkg/version>

I think more like:

>>> Build step
Building <pkg/version>
More logs ...

It depends on the context, when building a dependency the pkg/version is no longer necessary, it is already in a clear title category.

@perseoGI

Copy link
Copy Markdown
Contributor
>>> Build step
Building <pkg/version>
More logs ...

Looks promising!
Something that I'm a little bit concern (also in my previous POC), is if removing the pkg/version will cause users to lose some context of what is actually doing.

But I guess this is a price to pay in order to modernice and simplify the output.

A not an easy solution for avoiding loosing the context will be to use... tachaaan "rich spinners" 🤣

@memsharded

Copy link
Copy Markdown
Member Author

Something that I'm a little bit concern (also in my previous POC), is if removing the pkg/version will cause users to lose some context of what is actually doing.

I agree, this is why I think that this has to be moved forward step by step, checking the actual output UX, seeing that the context is clear, etc. From the snapshots above, it seems that information is not being lost, and the context is clear so far, and this initiative is mostly eliminating a lot of visual noise.

@perseoGI

Copy link
Copy Markdown
Contributor

From the snapshots above, it seems that information is not being lost, and the context is clear so far, and this initiative is mostly eliminating a lot of visual noise.

I agree that this initiative eliminates plenty of visual noise (which is what we were looking for)
I dont know if in larger outputs we may loose the scope. Lets continue with this migration and we will see.

@czoido
czoido merged commit 38bb27a into conan-io:develop2 Jun 22, 2026
18 checks passed
@AbrilRBS
AbrilRBS deleted the ux/conan_output_generate branch June 22, 2026 15:06
@memsharded memsharded added this to the 2.30.0 milestone Jun 22, 2026
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