chore: make ignore reason human readable in text output

[gg-mmill]

Context

Following #1024 , small fix to make ignore reasons human readable.

What has been done

Validation

PR check list

• As much as possible, the changes include tests (unit and/or functional)
• If the changes affect the end user (new feature, behavior change, bug fix) then the PR has a changelog entry (see doc/dev/getting-started.md). If the changes do not affect the end user, then the skip-changelog label has been added to the PR.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.05%. Comparing base (6d0d8b8) to head (aa8b108).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1031   +/-   ##
=======================================
  Coverage   92.05%   92.05%           
=======================================
  Files         181      181           
  Lines        7726     7728    +2     
=======================================
+ Hits         7112     7114    +2     
  Misses        614      614           

Flag	Coverage Δ
unittests	92.05% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[agateau-gg]

This change makes it hard to get a machine-friendly constant, which would be handy for machine-friendly outputs like JSON.

At the minimum I would remove the __str__() override: code that wants the human-readable version can call .value to get it.

But even this way using human-friendly values makes recreating the enum from a string more difficult:

class MyEnum(str, Enum):
    FOO = "This is foo"
    BAR = "This is bar"

# Fails
MyEnum("FOO")

# Fails
MyEnum("MyEnum.FOO")

# Works
MyEnum("This is foo")

That's why I would advocate for something like this instead:

class MyEnum(str, Enum):
    FOO = "FOO"
    BAR = "BAR"

    @property
    def display_value(self) -> str:
        # Or use a dict
        if self == self.FOO:
            return "This is foo"
        elif self == self.BAR:
            return "This is bar"
        raise ValueError(self)

With this we can do:

>>> MyEnum("FOO")
<MyEnum.FOO: 'FOO'>
>>> MyEnum.FOO.value
'FOO'
>>> MyEnum.FOO.display_value
'This is foo'