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

Fix typos and grammar in pom.xml #268

Merged
merged 2 commits into from
Sep 29, 2022
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 13 additions & 12 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@
additional JVMs. -->
<argLine>
<!-- The JVM arguments specified in `.mvn/jvm.config` also apply to
test JVMs, as those are also non-interactive processess running
test JVMs, as those are also non-interactive processes running
Error Prone, and thus similarly make extensive use of unsupported
`javac`-internal APIs, in some cases even reflectively inspecting
them. -->
Expand Down Expand Up @@ -110,11 +110,12 @@
-Dfile.encoding=${project.build.sourceEncoding}
<!-- This argument *can* be set through Surefire's
'systemPropertyVariables' configuration setting, but by placing it
here it automatically also applies to the Failsafe plugin. On Unix
systems we use a lower-quality source of randomness, so as to avoid
potential slowdown of tests relying on java.security.SecureRandom.
Note that it is not fatal for the file to not exist, so this
setting is Windows-compatible. The '/./' syntax is no accident; see
here it automatically also applies to the Failsafe plugin. To avoid
potential slowdown of tests relying on
`java.security.SecureRandom`, on Unix systems we use a
lower-quality source of randomness. Note that it is not fatal for
the file not to exist, so this setting is Windows-compatible. The
`/./` syntax is no accident; see
https://bugs.openjdk.java.net/browse/JDK-6202721 for details. -->
-Djava.security.egd=file:/dev/./urandom
<!-- On Mac OS X, running in headless mode prevents the forked JVMs
Expand All @@ -133,7 +134,7 @@
<error-prone.patch-args />
<error-prone.self-check-args />
<!-- The Maven `groupId` under which Error Prone dependencies are
published. By default we use an official Error Prone release. This
published. We use an official Error Prone release by default. This
property allows the `error-prone-fork` profile below to build the
project using Picnic's Error Prone fork instead. -->
<groupId.error-prone>com.google.errorprone</groupId.error-prone>
Expand Down Expand Up @@ -1320,7 +1321,7 @@
</dependencyManagement>
</profile>
<profile>
<!-- Error Prone checks that are not availabe from Maven Central;
<!-- Error Prone checks that are not available from Maven Central;
these are therefore not enabled by default. -->
<id>non-maven-central</id>
<properties>
Expand Down Expand Up @@ -1394,7 +1395,7 @@
</build>
</profile>
<profile>
<!-- By default we verify various aspects of a module and the
<!-- By default, we verify various aspects of a module and the
artifact(s) it produces. We define these checks in a profile so
that they can be disabled during development. See also the
`error-prone` profile defined below. -->
Expand Down Expand Up @@ -1591,9 +1592,9 @@
</properties>
</profile>
<profile>
<!-- The `build-checks` and `error-prone` profiles enable a whole
bunch of additional compile checks. By default those warnings break
the build. This profile allows one to collect all build warnings
<!-- The `build-checks` and `error-prone` profiles enable a bunch
of additional compile checks. By default, those warnings break the
build. This profile allows one to collect all build warnings
without failing the build. -->
<id>disallow-warnings</id>
<activation>
Expand Down