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

maint: bump deps #596

Merged
merged 10 commits into from
Jun 21, 2024
Merged

maint: bump deps #596

merged 10 commits into from
Jun 21, 2024

Conversation

lread
Copy link
Collaborator

@lread lread commented Jun 20, 2024

Please complete and include the following checklist:

  • I have read CONTRIBUTING and the Etaoin Developer Guide.

  • This PR corresponds to an issue that the Etaoin maintainers have agreed to address.

  • This PR contains test(s) to protect against future regressions

  • I have updated CHANGELOG.adoc with a description of the addressed issue.

Jvm nrepl no longer works with clojure 1.9
Our docs introduce Etaoin with a trip to wikipedia.
Our docs are tested with test-doc-blocks.
Wikipedia changed a wee bit.
Which made our doc tests fail.
Adapt.
This step was skipped on test failure. Ironicaly that's the only time I
think I really currently need it!
@lread
Copy link
Collaborator Author

lread commented Jun 20, 2024

Well on the positive side I learned about uploading artifacts on github actions!
CI now is setup to save any ./target/debug-screenshots/* as an artifact.

Something is amiss with our Wikipedia tests. We wait for the search input and I temporarily dumped screenshot. Here it is:
wikipedia-wait-failure

It is really not the screen I was expecting... It is not the one I see locally on first hitting Wikipedia:
image

And... there is no search input.

@lread
Copy link
Collaborator Author

lread commented Jun 20, 2024

My Mama always told me, "Never test against changing websites, son; it makes for very brittle tests." But I still like the fact that an example in our user guide hits Wikipedia, so I might suffer the occasional breakage. Maybe Wikipedia is rolling out something new as I write this. I'll give it a day and retry.

@lread
Copy link
Collaborator Author

lread commented Jun 21, 2024

Ah... well, one thing that is probably happening is that the mobile-friendly view is being presented when hit from CI. Maybe that's most of the issue for the failure.

Small thing, but sorts jdk8 before jdk11 which makes github actions workflow
job listing easier to follow.
@lread lread merged commit c5054aa into master Jun 21, 2024
53 checks passed
@lread lread deleted the lread-maint-2024-06-20 branch July 28, 2024 18:13
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.

1 participant