Skip to content

Conversation

@Andersson007
Copy link
Contributor

SUMMARY

README: add Communication section

ISSUE TYPE
  • Docs Pull Request

@Andersson007
Copy link
Contributor Author

cc @oraNod

@packit-as-a-service

This comment was marked as outdated.

@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

@webknjaz webknjaz force-pushed the irc_remnants branch 2 times, most recently from 466b9fc to f053875 Compare September 11, 2024 12:03
@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

Andersson007 and others added 8 commits September 12, 2024 15:08
Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Don Naro <dnaro@redhat.com>
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
@sonarqubecloud
Copy link

@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
@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