readthedocs / actions

GitHub Actions for Read the Docs
MIT License
33 stars 10 forks source link

Change the verbose name of the action #11

Closed agjohnson closed 2 years ago

agjohnson commented 2 years ago

Multiple times now, I've surprised myself and tried to click the check for the PR docs links action in the PR list of checks. Apparently I'm confused by how prominent the check looks, but of course the "details" link for that check does not do what you expect.

I'm actually not even sure where this name comes from, but just assuming its centrally configured here somehow.

So, I would say this first check is more prominent that our own RTD build check:

sorry, github is broken

And even though our check has our icon in that list, I still somehow want to click the first check, the check for the links. So, if I am actively/subconsciously doing this as someone familiar with RTD, there must be others that would notice this.

Looking more at this, "Read the Docs" looks more official than "docs/readthedocs.com:...", and the first link mentions "pull request preview". Though the links are in the PR description, users are probably still used to clicking the check detail links, so would be bound to hit this confusion.

This is a case where showing this action in the PR action list is probably a worse scenario, but that's up to GitHub to deal with.

But because we shouldn't expect users to interact with this particular action in the list of checks, I think a much less prominent name would at least not overwhelm our actual PR check.

We could follow the naming scheme used in other checks, this is definitely less prominent and doesn't make me want to keep clicking it.

great job github

humitos commented 2 years ago

I'm actually not even sure where this name comes from, but just assuming its centrally configured here somehow.

The names comes from https://github.com/readthedocs/actions/blob/main/.github/workflows/readthedocs-preview.yaml#L1

It's a combination of:

So, we can update our example to make it simpler --but it's per-repository, so it won't change the name from repositories already using the action.

Docs: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#name

humitos commented 2 years ago

I tweaked it a little following your suggestion and ended up with the following (see https://github.com/readthedocs/actions/pull/13):

Screenshot_2022-08-18_10-45-23

If you are happy with it, we should change the documentation at https://github.com/readthedocs/actions/tree/main/preview#how-to-use-it as well. Let me know!

agjohnson commented 2 years ago

Ah yeah, looks much better! I don't think I'd click on that first

humitos commented 2 years ago

Cool! Closed by https://github.com/readthedocs/actions/commit/f132761555725a8a8ed40ff29d7de22522b096b4