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

add wildcard example for proxyjump - also change user in command prompt in other examples #207

Merged
merged 2 commits into from
Feb 5, 2025

Conversation

alaniwi
Copy link
Contributor

@alaniwi alaniwi commented Feb 5, 2025

I suggest that @amanning9 reviews this PR.

@alaniwi alaniwi requested a review from amanning9 February 5, 2025 16:05
clarify description
Copy link
Member

@amanning9 amanning9 left a comment

Choose a reason for hiding this comment

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

Looks good to me. Not sure if the bit about canonical names / search paths is really required, probably most users won't have done this.

But happy either way!

(NB in the process of reviewing I pulled and built the site, so confirmed it's good from a hugo/looks point of view too!)

@alaniwi
Copy link
Contributor Author

alaniwi commented Feb 5, 2025

@amanning9 Thanks, will merge.

@alaniwi alaniwi merged commit e7a43a3 into main Feb 5, 2025
1 check passed
@alaniwi alaniwi deleted the proxyjump_wildcard branch February 6, 2025 10:25
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