eza-community / eza

A modern, maintained replacement for ls
https://eza.rocks
MIT License
8.79k stars 170 forks source link

REFACTOR: Improve branch visibility. #933

Closed allaboutevemirolive closed 1 month ago

allaboutevemirolive commented 2 months ago

This PR improves branch visibility by adding space immediately in the ascii_art method rather than deciding whether to add space or not at runtime. This branch visibility is based on how the original Tree built the branch to make it look better.

Before:

before_branch

After:

after_branch

PThorpe92 commented 2 months ago

Thanks for the PR!

Personally I like it, it makes sense to me. I'm for merging if everyone else is.

daviessm commented 2 months ago

Do we need to worry about the failed FreeBSD check?

PThorpe92 commented 2 months ago

image this looks like the same exact issue from last week. I thought we merged a fix for this?

EDIT: yup it was #929

OK I see the other issue is the tryCMD test output. image Because the output is different (space added) those tests are going to have to be adjusted. @cafkafk or @MartinFillon would know the procedure here with new output.

allaboutevemirolive commented 2 months ago

As per the tests, just run just idump then do just itest you will need a working nix shell, and Just

Hi @MartinFillon, just to confirm, do I need to run those tests on my local machine?

I ran just build-release test-release, but the command just idump followed by just itest is not working.

MartinFillon commented 2 months ago

yup on your local machine and look for any ptests_file updated other can be dicarded iirc

allaboutevemirolive commented 2 months ago

I'm not sure why cicd didn't pass. Here I put updated files after run idump.

git status $ git status On branch branch Changes not staged for commit: (use "git add/rm ..." to update what will be committed) (use "git restore ..." to discard changes in working directory) modified: tests/cmd/absolute_recurse_unix.stdout modified: tests/cmd/absolute_unix.stdout deleted: tests/gen/absolute_recurse_unix.stderr deleted: tests/gen/absolute_recurse_unix.stdout deleted: tests/gen/absolute_unix.stderr deleted: tests/gen/absolute_unix.stdout modified: tests/gen/tree_long_unix.stdout modified: tests/gen/tree_unix.stdout modified: tests/ptests/ptest_10c50228fc1c6107.stdout modified: tests/ptests/ptest_132ac9f159c3473e.stdout modified: tests/ptests/ptest_1ab8279d631495b0.stdout modified: tests/ptests/ptest_1e61d8c305ff6c48.stdout modified: tests/ptests/ptest_2176b1b061cb7da2.stdout modified: tests/ptests/ptest_298b768f6bb48dac.stdout modified: tests/ptests/ptest_3259dc201df8e734.stdout modified: tests/ptests/ptest_41b5466dabf23647.stdout modified: tests/ptests/ptest_469e79a86c2c874f.stdout modified: tests/ptests/ptest_46a8435d247c34c9.stdout modified: tests/ptests/ptest_4b0ed60c44c669f.stdout modified: tests/ptests/ptest_6e6893c2c2254622.stdout modified: tests/ptests/ptest_72f8f65472fcd4fa.stdout modified: tests/ptests/ptest_7d1cd636566df8cd.stdout modified: tests/ptests/ptest_825e60e73630f857.stdout modified: tests/ptests/ptest_8484317bd3add4dc.stdout modified: tests/ptests/ptest_8becd3030ba5621c.stdout modified: tests/ptests/ptest_8ece6263396deeab.stdout modified: tests/ptests/ptest_abc83ec759ddab6.stdout modified: tests/ptests/ptest_bbfc26a93fbe15a7.stdout modified: tests/ptests/ptest_c4ead86a12b992a8.stdout modified: tests/ptests/ptest_caa4e824b02fa569.stdout modified: tests/ptests/ptest_cc209e82fa5f78eb.stdout modified: tests/ptests/ptest_cee646e8f21997ad.stdout modified: tests/ptests/ptest_de916a0d5867def3.stdout modified: tests/ptests/ptest_e1033830f4720a6f.stdout modified: tests/ptests/ptest_f261ab10a0ea20f.stdout modified: tests/ptests/ptest_ffbdb9af2de10fa.stdout no changes added to commit (use "git add" and/or "git commit -a")

Then I run git add tests/ptests/* to only add ptest.

MartinFillon commented 2 months ago

Gonna take a look at your branch if you allow me

cafkafk commented 1 month ago

From #974

If this PR passes the CICD, I will close PR #933.

Review/pr continues in #974