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

minor updates to the 'From Source' page #1222

Merged
merged 1 commit into from
Feb 5, 2024
Merged

Conversation

kenjenkins
Copy link
Contributor

The part about "a wide array of architectures and operating systems" predates the dependency on Envoy. Currently Pomerium is limited by the OSes/architectures supported by Envoy, which I believe includes only Linux and macOS, for amd64 and arm64. (According to an announcement at https://groups.google.com/g/envoy-announce/c/2e-mSnit_BQ/m/VvweUrdaAAAJ Windows is no longer supported after Envoy 1.27.)

As of pomerium/pomerium@faa2a86 golangci-lint is no longer required to be installed locally (it is instead invoked via go run).

Finally, make sure "Pomerium" is capitalized consistently.

The part about "a wide array of architectures and operating systems"
predates the dependency on Envoy. Currently Pomerium is limited by the
OSes/architectures supported by Envoy, which I believe includes only
Linux and macOS, for amd64 and arm64. (According to an announcement at
https://groups.google.com/g/envoy-announce/c/2e-mSnit_BQ/m/VvweUrdaAAAJ
Windows is no longer supported after Envoy 1.27.)

As of pomerium/pomerium@faa2a86
golangci-lint is no longer required to be installed locally (it is
instead invoked via `go run`).

Finally, make sure "Pomerium" is capitalized consistently.
@kenjenkins kenjenkins requested a review from ZPain8464 February 5, 2024 21:50
@kenjenkins kenjenkins requested a review from a team as a code owner February 5, 2024 21:50
Copy link

netlify bot commented Feb 5, 2024

Deploy Preview for pomerium-docs ready!

Name Link
🔨 Latest commit 9cf5d19
🔍 Latest deploy log https://app.netlify.com/sites/pomerium-docs/deploys/65c158430f06780008d1910d
😎 Deploy Preview https://deploy-preview-1222--pomerium-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ZPain8464
Copy link
Contributor

ZPain8464 commented Feb 5, 2024

@kenjenkins thanks for updating this page :)

@kenjenkins kenjenkins merged commit 51d47cb into main Feb 5, 2024
7 checks passed
@kenjenkins kenjenkins deleted the kenjenkins/from-source branch February 5, 2024 22:27
backport-actions-token bot pushed a commit that referenced this pull request Feb 5, 2024
The part about "a wide array of architectures and operating systems"
predates the dependency on Envoy. Currently Pomerium is limited by the
OSes/architectures supported by Envoy, which I believe includes only
Linux and macOS, for amd64 and arm64. (According to an announcement at
https://groups.google.com/g/envoy-announce/c/2e-mSnit_BQ/m/VvweUrdaAAAJ
Windows is no longer supported after Envoy 1.27.)

As of pomerium/pomerium@faa2a86
golangci-lint is no longer required to be installed locally (it is
instead invoked via `go run`).

Finally, make sure "Pomerium" is capitalized consistently.
kenjenkins added a commit that referenced this pull request Feb 6, 2024
minor updates to the 'From Source' page (#1222)

The part about "a wide array of architectures and operating systems"
predates the dependency on Envoy. Currently Pomerium is limited by the
OSes/architectures supported by Envoy, which I believe includes only
Linux and macOS, for amd64 and arm64. (According to an announcement at
https://groups.google.com/g/envoy-announce/c/2e-mSnit_BQ/m/VvweUrdaAAAJ
Windows is no longer supported after Envoy 1.27.)

As of pomerium/pomerium@faa2a86
golangci-lint is no longer required to be installed locally (it is
instead invoked via `go run`).

Finally, make sure "Pomerium" is capitalized consistently.

Co-authored-by: Kenneth Jenkins <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants