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

Docsite: add Community guide #177

Merged
merged 10 commits into from
Sep 9, 2024

Conversation

Andersson007
Copy link
Contributor

Dear maintainers,
As a part of the Consolidating Ansible discussion platforms initiative, this PR adds the communication section template defined by the community to the README and the docsite. Similar PRs are now being raised across all the Ansible ecosystem projects.

Thank you

@Andersson007
Copy link
Contributor Author

ready for review

Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

docs/community.md Outdated Show resolved Hide resolved
docs/community.md Outdated Show resolved Hide resolved
docs/index.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

You also need to mention documentation.md in mkdocs.yml's nav list.

docs/index.md Outdated Show resolved Hide resolved
Co-authored-by: Felix Fontein <felix@fontein.de>
@Andersson007 Andersson007 marked this pull request as draft September 3, 2024 05:43
@Andersson007 Andersson007 marked this pull request as ready for review September 3, 2024 05:57
@Andersson007
Copy link
Contributor Author

@felixfontein thanks for reviewing! All done, PTAL

docs/community.md Outdated Show resolved Hide resolved
docs/community.md Outdated Show resolved Hide resolved
Andersson007 and others added 2 commits September 6, 2024 08:32
Co-authored-by: Felix Fontein <felix@fontein.de>
Co-authored-by: Felix Fontein <felix@fontein.de>
@Andersson007
Copy link
Contributor Author

@felixfontein committed, lgtm, thanks, PTAL

Copy link
Contributor

@samccann samccann left a comment

Choose a reason for hiding this comment

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

LGTM

docs/index.md Show resolved Hide resolved
Copy link
Contributor

@samccann samccann left a comment

Choose a reason for hiding this comment

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

one minor nit

@samccann samccann self-requested a review September 6, 2024 16:12
docs/index.md Outdated Show resolved Hide resolved
@Andersson007 Andersson007 merged commit c2f962f into ansible-community:main Sep 9, 2024
5 checks passed
@Andersson007
Copy link
Contributor Author

@felixfontein @samccann thanks for reviewing!

@felixfontein
Copy link
Collaborator

@Andersson007 thanks for implementing this (and the other PR)!

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