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

e2e tests: a small step towards arm and docs tweaks #3617

Merged
merged 2 commits into from
Apr 13, 2021
Merged

Conversation

goodhoko
Copy link
Contributor

@goodhoko goodhoko commented Apr 9, 2021

A few last days I spent trying to run e2e tests on my M1 Mac. This PR brings 1) a small patch that brings us closer to that 2) Some minor docs tweaks.

This alone isn't enough to make the e2e tests run on arm machines, that will need far more work. I decided to not spend time on that now, deffered the effort to this issue and pushed what I got so far with this PR.

RBC (review by commits) & DNS (do not squash) please .)

goodhoko added 2 commits April 9, 2021 13:11
- Make newlines after heading consistent.
- Make navigation notation consistent.
- Use simpler code formatting, keep double backticks only where needed.
- Explain M1 Mac situation and link to corresponding issue.
- Reword a sentence to follow natural time sequence.
- Remove dot prefix from relative paths
    Paths with the dot appear as relative to the current file (e2e-web.md)
    but that's wrong. Remove the dot from relative paths forcing the reader
    to figure out what the path is relative to. Also note it at top of the
    manual.
1.4 release doesn't provide builds for arm64. Bump to the latest version
(1.12) which does.
@goodhoko goodhoko requested a review from tsusanka April 9, 2021 11:13
@goodhoko
Copy link
Contributor Author

goodhoko commented Apr 9, 2021

The failing job doesn't seem to be related to changes in this PR. Might that be a temporary error fixable by a mere retry?

@tsusanka
Copy link
Contributor

tsusanka commented Apr 9, 2021

I've restarted and it seems to me that.

Copy link
Contributor

@tsusanka tsusanka left a comment

Choose a reason for hiding this comment

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

Mostly nit-picks, LGTM.

@tsusanka tsusanka merged commit 9e8c13a into develop Apr 13, 2021
@tsusanka tsusanka deleted the docs/test-docs branch April 13, 2021 06:10
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.

2 participants