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

note about h2spec and Mac Silicon #264

Merged
merged 1 commit into from
Jul 24, 2023
Merged

Conversation

pjfanning
Copy link
Contributor

relates to #262 and #218

@@ -35,6 +35,13 @@ The documentation is available at
- Make sure you have [sbt](https://www.scala-sbt.org/) installed.
- [Graphviz](https://graphviz.gitlab.io/download/) is needed for the scaladoc generation build task, which is part of the release.

### h2spec
Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn't this section make more sense in CONTRIBUTING.mg, as it's a detail only relevant to people building/testing the project and not users.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This affects anyone who runs from tests on a Mac Silicon. Not necessarily a contributor.

Copy link
Contributor

@mdedetrich mdedetrich left a comment

Choose a reason for hiding this comment

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

lgtm

@pjfanning pjfanning merged commit d604de6 into apache:main Jul 24, 2023
10 checks passed
@pjfanning pjfanning deleted the h2spec-note branch July 24, 2023 07:56
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