Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Various fixes for awesome.md #4448

Merged
merged 5 commits into from
Nov 25, 2024
Merged

Various fixes for awesome.md #4448

merged 5 commits into from
Nov 25, 2024

Conversation

pat-s
Copy link
Contributor

@pat-s pat-s commented Nov 23, 2024

  • git.rustybever cannot be reached. Might be temporary
  • wording updates

@pat-s pat-s added the documentation docu & docs label Nov 23, 2024
@woodpecker-bot
Copy link
Contributor

woodpecker-bot commented Nov 23, 2024

Deployment of preview was torn down

docs/src/pages/awesome.md Outdated Show resolved Hide resolved
@xoxys
Copy link
Member

xoxys commented Nov 23, 2024

Not sure how we want to handle (temp) unavailable resources.

Co-authored-by: Robert Kaussow <mail@thegeeklab.de>
@pat-s
Copy link
Contributor Author

pat-s commented Nov 24, 2024

Maybe having a waiting period for ~ 3 days makes sense? It surely differs for the domain at hand, e.g. I'd expect a git.something to be back earlier than a random personal website domain.

@xoxys
Copy link
Member

xoxys commented Nov 24, 2024

Assumptions aside we should find a generic rule. I would wait at least a week.

@pat-s
Copy link
Contributor Author

pat-s commented Nov 24, 2024

The problem is that within that week all CI checks will fail without any action.

And this causes interruption for automated processed and also takes developer time looking at runs and causes unneeded discussions about the failure per se.

@xoxys
Copy link
Member

xoxys commented Nov 24, 2024

Move linkchecking of communit resources to a dedicated workflow as we do for security scanning and make it not required.

@pat-s
Copy link
Contributor Author

pat-s commented Nov 24, 2024

Good ideas overall.

The check is already optional.

To not block this PR due to this, I've added the resources again, so this PR is not blocked by this discussion.

@6543 6543 merged commit ceee2a0 into main Nov 25, 2024
7 checks passed
@6543 6543 deleted the docs/awesome-fixes branch November 25, 2024 15:15
@woodpecker-bot woodpecker-bot mentioned this pull request Nov 25, 2024
1 task
@woodpecker-bot woodpecker-bot mentioned this pull request Dec 14, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation docu & docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants