Print USC compilation details

[ddoktorski]

Closes #2403

Introduced changes

• Print information about compiling Sierra as a spinner message
• Skip printing the path for unsaved files

Checklist

• Linked relevant issue
• Updated relevant documentation
• Added relevant tests
• Performed self-review of the code
• Added changes to CHANGELOG.md

[kkawula]

Spinner looks cool, but when compilation takes milliseconds it appears and disappears as fast we can't read it. So maybe to inform user whats going on, we should add heading about compilation?
Sth like:

Compiling Sierra to Casm...
<spinner>

[franciszekjob]

Spinner looks cool, but when compilation takes milliseconds it appears and disappears as fast we can't read it. So maybe to inform user whats going on, we should add heading about compilation? Sth like:

Compiling Sierra to Casm...
<spinner>

Valid point, it could be added to simply serve an information for user. Ofc it would disappear with spinner as well.

[franciszekjob]

We can think about adding a simple message that would disappear with spinner as well but this is not necessary.

Overall, I like this 👍 .

[cptartur]

We can think about adding a simple message that would disappear with spinner as well but this is not necessary.

Overall, I like this 👍 .

@franciszekjob if it's going to disappear then I don't think it would change much in terms of readability

[franciszekjob]

We can think about adding a simple message that would disappear with spinner as well but this is not necessary.
Overall, I like this 👍 .

@franciszekjob if it's going to disappear then I don't think it would change much in terms of readability

@cptartur I don't refer to situation when compilation is very quick. The clue of this issue is that we want to prevent the user from thinking that something is broken when compilation takes longer. Message will be helpful the way that spinner indicates Sierra to CASM compilation and not something else. not valid anymore

[franciszekjob]

We can think about adding #2631 (review) that would disappear with spinner as well but this is not necessary.

Nevermind, I didn't notice that the message was already added after @kkawula comment 😅 , all fine.