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

Testing link-checker-gpt 0.1.0-beta #13

Merged
merged 37 commits into from
Aug 21, 2023
Merged

Testing link-checker-gpt 0.1.0-beta #13

merged 37 commits into from
Aug 21, 2023

Conversation

kingdonb
Copy link
Owner

@kingdonb kingdonb commented Aug 15, 2023

Fix #15

Unsure if we can use v0 prerelease versioning for a GitHub action.

May just promote v1 if everything seems to be working. Still need to finish testing:

There needs to be a PR so we can see the test workflow executing.

It will still fail, because we are waiting for this one and its friends to merge:

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Kingdon Barrett added 3 commits August 17, 2023 10:02
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb kingdonb mentioned this pull request Aug 17, 2023
Kingdon Barrett added 12 commits August 17, 2023 17:54
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb
Copy link
Owner Author

Let's give it a configurable URL, configure it to check for a healthy sitemap and configurable timeout

We can clarify in the README how to adopt it in your repo, what options it has, and test it in a repository for some other domain. Those things shouldn't be very difficult to add.

https://developer.fermyon.com/bartholomew/seo#generating-a-sitemap

(We have one of these blogs and we can add a sitemap to it to make it work with this, it's recommended anyway and I haven't done it yet...)

Kingdon Barrett added 11 commits August 18, 2023 08:49
Move the workflow to root directory, following:
https://docs.github.com/en/actions/creating-actions/about-custom-actions#good-practices-for-release-management

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
preserve it for posterity in a separate .yml

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
but we can tell it to try again and keep polling until timeout or the
PREVIEW_URL site becomes ready.

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Kingdon Barrett added 9 commits August 21, 2023 12:47
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
the advice to use env is for scripts that users embed in their own repos
https://docs.github.com/en/actions/using-workflows/using-github-cli-in-workflows

we need to take githubToken as an input, evidently

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
Signed-off-by: Kingdon Barrett <kingdon@weave.works>
not sure which of these or if both are effective, but let's see if this
breaks anything (it ought not!)

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb kingdonb changed the title Testing link-checker-gpt 0.1.0-pre Testing link-checker-gpt 0.1.0-beta Aug 21, 2023
@kingdonb
Copy link
Owner Author

The v1-beta branch has accomplished all the goals of #15.

@kingdonb
Copy link
Owner Author

I will close v1-beta branch and make a tag. The merged commit will get both tags: v1-beta and v0.1.0-beta.1 - the Docker action needs one small change, so that tags are built rather than just the v1-beta branch.

Signed-off-by: Kingdon Barrett <kingdon@weave.works>
@kingdonb kingdonb merged commit 9212eb0 into main Aug 21, 2023
1 check failed
@kingdonb kingdonb deleted the test-prerelease branch August 21, 2023 18:18
This pull request was closed.
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.

Operationalize this repo
1 participant