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

ci: add script to check modules on registry.coder.com #340

Merged
merged 3 commits into from
Nov 15, 2024

Conversation

johnstcn
Copy link
Member

I made a simple and dumb script to check the modules on registry.coder.com.
Gonna see how it goes for the moment before hooking it up to any sad trombones.

@johnstcn johnstcn self-assigned this Nov 14, 2024
Copy link

@dannykopping dannykopping left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

check.sh Outdated
printf "=== Check module %s at %s\n" "${module}" "${url}"
status_code=$(curl --output /dev/null --head --silent --fail --location "${url}" --retry 3 --write-out "%{http_code}")
# shellcheck disable=SC2181
if (( status_code != 200 )); then

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if the server returns 3XX cache responses?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't noticed any of those after running it but I suppose the proof is in the eating.

Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few suggestions but LGTM.

.github/workflows/check.yaml Outdated Show resolved Hide resolved
check.sh Outdated Show resolved Hide resolved
@johnstcn johnstcn merged commit 57d96ca into main Nov 15, 2024
2 checks passed
@johnstcn johnstcn deleted the cj/check-modules branch November 15, 2024 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants