Skip to content

apiage: Do not leave empty sections for packages in markdown#753

Merged
mergify[bot] merged 1 commit intoceph:masterfrom
anoopcs9:fix-empty-pkg-hdr
Aug 12, 2022
Merged

apiage: Do not leave empty sections for packages in markdown#753
mergify[bot] merged 1 commit intoceph:masterfrom
anoopcs9:fix-empty-pkg-hdr

Conversation

@anoopcs9
Copy link
Collaborator

@anoopcs9 anoopcs9 commented Aug 6, 2022

For those packages without any preview/deprecated APIs we used to include a header despite any content under it. We could instead mention the state in a meaningful sentence.

Fixes: #595

@anoopcs9 anoopcs9 force-pushed the fix-empty-pkg-hdr branch 2 times, most recently from 3cebbc4 to bf5d4fd Compare August 6, 2022 09:23
@anoopcs9 anoopcs9 requested a review from phlogistonjohn August 8, 2022 16:53
@anoopcs9 anoopcs9 force-pushed the fix-empty-pkg-hdr branch 2 times, most recently from 07c97ef to fcc5ce4 Compare August 9, 2022 15:39
Copy link
Collaborator

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

Change looks great. However we need to resolve the outstanding CI issue with revive prior to merging.

@anoopcs9
Copy link
Collaborator Author

@Mergifyio rebase

For those packages without any preview/deprecated APIs we used to
include a header despite any content under it. We could instead
mention the state in a meaningful sentence.

Signed-off-by: Anoop C S <anoopcs@cryptolab.net>
@mergify
Copy link

mergify bot commented Aug 12, 2022

rebase

✅ Branch has been successfully rebased

@anoopcs9 anoopcs9 added the no-API This PR does not include any changes to the public API of a go-ceph package label Aug 12, 2022
@mergify mergify bot merged commit a79b461 into ceph:master Aug 12, 2022
@anoopcs9 anoopcs9 deleted the fix-empty-pkg-hdr branch August 12, 2022 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-API This PR does not include any changes to the public API of a go-ceph package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

apiage: packages without deprecated/preview apis look odd

2 participants