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

Issue152 - this issue's problem was actually already resolved but in reading - some cleanup of text seemed proper. #215

Merged
merged 6 commits into from
Jan 6, 2025

Conversation

morrowc
Copy link
Contributor

@morrowc morrowc commented Dec 28, 2024

Cleanup some of the text in the readme, surrounding rotation content.

Add some clarification that SSL Profiles maybe used for:
  * client services (services on a system which initiate connections)
  * server services (services on a system which receive connections)
  * both (clients and servers)
Cleanup a bit of the README content, mostly
language fixes seen upon re-reading the readings.
@morrowc morrowc requested a review from marcushines December 28, 2024 19:42
certz/README.md Outdated Show resolved Hide resolved
@morrowc morrowc requested a review from marcushines January 4, 2025 00:11
@morrowc
Copy link
Contributor Author

morrowc commented Jan 4, 2025

Oddly the build of: //...
seems to (minus a crapload of logging?) finish just fine on my local build attempts.

$ go version
go version go1.23.4 linux/amd64
$ bazel version
Build label: 6.4.0
Build target: bazel-out/k8-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Thu Oct 19 17:07:43 2023 (1697735263)
Build timestamp: 1697735263
Build timestamp as int: 1697735263

I can try with bazel 7.4.1 to be sure.

@morrowc
Copy link
Contributor Author

morrowc commented Jan 4, 2025

Yes, builds locally with 6.4 and 7.4 both work fine. #confused

Make this BUILD target look like a working BUILD target (credentialz)
@marcushines marcushines merged commit 8b3e603 into main Jan 6, 2025
4 checks passed
@marcushines marcushines deleted the Issue152 branch January 6, 2025 16:01
morrowc added a commit that referenced this pull request Jan 22, 2025
…reading - some cleanup of text seemed proper. (#215)

* Issue202: Add clarification on usage of ssl profiles.

Add some clarification that SSL Profiles maybe used for:
  * client services (services on a system which initiate connections)
  * server services (services on a system which receive connections)
  * both (clients and servers)

* Documentation Cleanup.

Cleanup a bit of the README content, mostly
language fixes seen upon re-reading the readings.

* Address comment.

* Attempting to fix a broken bazel build in github-actions.

Make this BUILD target look like a working BUILD target (credentialz)

* Add to the BUILD fixing?
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