Skip to content

Add release step for updating algolia index#805

Merged
jetstack-bot merged 1 commit intocert-manager:masterfrom
SgtCoDFish:algolia-release
Jan 24, 2022
Merged

Add release step for updating algolia index#805
jetstack-bot merged 1 commit intocert-manager:masterfrom
SgtCoDFish:algolia-release

Conversation

@SgtCoDFish
Copy link
Member

@SgtCoDFish SgtCoDFish commented Jan 24, 2022

We've missed this in every release since 0.15 so now might be a good time to start 😁

@SgtCoDFish
add release step for updating algolia index
4988804 
Signed-off-by: Ashley Davis <ashley.davis@jetstack.io>
@jetstack-bot jetstack-bot added dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Jan 24, 2022
@netlify
Copy link

netlify bot commented Jan 24, 2022

✔️ Deploy Preview for cert-manager-website ready!

🔨 Explore the source changes: 4988804

🔍 Inspect the deploy log: https://app.netlify.com/sites/cert-manager-website/deploys/61eebd84bc5dc500085c0ab0

😎 Browse the preview: https://deploy-preview-805--cert-manager-website.netlify.app/docs/contributing/release-process

jakexks
jakexks approved these changes Jan 24, 2022
View reviewed changes
Copy link
Member

@jakexks jakexks left a comment

Choose a reason for hiding this comment

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

/lgtm

@JoshVanL
Copy link
Contributor

JoshVanL commented Jan 24, 2022

/lgtm

@jetstack-bot jetstack-bot added the lgtm Indicates that a PR is ready to be merged. label Jan 24, 2022
@jetstack-bot
Copy link
Contributor

jetstack-bot commented Jan 24, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jakexks, SgtCoDFish

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jetstack-bot jetstack-bot merged commit 36915e1 into cert-manager:master Jan 24, 2022
@SgtCoDFish SgtCoDFish deleted the algolia-release branch January 24, 2022 15:03
munnerz
munnerz reviewed Jan 24, 2022
View reviewed changes
content/en/docs/contributing/release-process.md

9. Open a PR against our
[Algolia indexing configuration](https://github.com/algolia/docsearch-configs/blob/master/configs/cert-manager.json#L7-L13)
including the new version for search indexing, as in
Copy link
Member

@munnerz munnerz Jan 24, 2022

Choose a reason for hiding this comment

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

It's actually the old version that we need to include here (e.g. when v1.7 is release, v1.6 should be added to that list)

This was referenced Jan 26, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@munnerz munnerz munnerz left review comments

@jakexks jakexks jakexks approved these changes

Assignees

@jakexks jakexks

@JoshVanL JoshVanL

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates that all commits in the pull request have the valid DCO sign-off message. lgtm Indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@SgtCoDFish @JoshVanL @jetstack-bot @munnerz @jakexks