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 a Communication section to README #640

Merged
merged 9 commits into from
Sep 12, 2024

Conversation

Andersson007
Copy link
Contributor

SUMMARY

README: add Communication section

ISSUE TYPE
  • Docs Pull Request

@Andersson007
Copy link
Contributor Author

cc @oraNod

This comment was marked as outdated.

README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
@Andersson007
Copy link
Contributor Author

@oraNod committed, thanks, PTAL

Copy link
Member

@webknjaz webknjaz left a comment

Choose a reason for hiding this comment

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

One of the blocking checks is the missing changelog fragment. Add one, following the surrounding style in https://ansible-pylibssh.rtfd.io/en/latest/changelog/ (past tense et al). Here's an example: 4bd4911#diff-b3c6051ecd8e19ab34e671511fc5e12b70d621bc12dc59d6b8268171fd827f22.

See https://ansible-pylibssh.rtfd.io/en/latest/contributing/guidelines/#adding-change-notes-with-your-prs

README.rst Outdated Show resolved Hide resolved
@webknjaz webknjaz force-pushed the irc_remnants branch 2 times, most recently from 466b9fc to f053875 Compare September 11, 2024 12:03
README.rst Outdated Show resolved Hide resolved
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided There is a change note present in this PR label Sep 11, 2024
@Andersson007
Copy link
Contributor Author

the changelog fragment is there, hope it's OK, thanks

Copy link

sonarcloud bot commented Sep 12, 2024

@webknjaz webknjaz changed the title README: add Communication section Add a Communication section to README Sep 12, 2024
@webknjaz webknjaz enabled auto-merge (squash) September 12, 2024 13:18
@webknjaz webknjaz merged commit eee08ab into ansible:devel Sep 12, 2024
120 checks passed
@Andersson007
Copy link
Contributor Author

@oraNod @Ruchip16 @webknjaz thanks for reviewing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:provided There is a change note present in this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants