na4zagin3 / satyrographos

Package manager for SATySFi
GNU Lesser General Public License v3.0
44 stars 13 forks source link

Fix indents in compatibility warning messages #164

Closed matsud224 closed 3 years ago

matsud224 commented 3 years ago

This PR fixes #86 like below message:

Installation completed!

Compatibility notice for library assert-eq:

  Packages have been renamed.

    assert-eq.satyg -> assert-eq/assert-eq.satyg

Compatibility notice for library class-exdesign:

  Packages have been renamed.

    article-ja.satyh -> class-exdesign/article-ja.satyh
    exdesign.satyh -> class-exdesign/exdesign.satyh

Compatibility notice for library class-slydifi:

  Packages have been renamed.

    functions/color.satyh -> class-slydifi/functions/color.satyh
    functions/figbox.satyh -> class-slydifi/functions/figbox.satyh
    functions/footnote.satyh -> class-slydifi/functions/footnote.satyh
    functions/overlay.satyh -> class-slydifi/functions/overlay.satyh
    functions/param.satyh -> class-slydifi/functions/param.satyh
    slydifi.satyh -> class-slydifi/slydifi.satyh
    theme/akasaka.satyh -> class-slydifi/theme/akasaka.satyh
    theme/hakodate.satyh -> class-slydifi/theme/hakodate.satyh
    theme/plain.satyh -> class-slydifi/theme/plain.satyh
...
na4zagin3 commented 3 years ago

@matsud224 thank you very much!!! Can you please update the unit tests with dune test and then dune promote?

matsud224 commented 3 years ago

I ran dune test and dune promote then committed. But CI test still fails...

na4zagin3 commented 3 years ago

@matsud224, I'm sorry that the test cases were locale-dependent. I just fixed them with https://github.com/na4zagin3/satyrographos/pull/163. Do you mind to rebase onto the current master and run dune test and dune promote again?

na4zagin3 commented 3 years ago

Okay, now unit tests pass. Thank you!!