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

Remove family provider migrator docs #216

Merged
merged 1 commit into from
Jun 12, 2023

Conversation

sergenyalcin
Copy link
Member

Description of your changes

Remove family provider migrator docs because they moved to the new repo

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR if necessary.

Signed-off-by: Sergen Yalçın <yalcinsergen97@gmail.com>
Copy link
Collaborator

@ulucinar ulucinar left a comment

Choose a reason for hiding this comment

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

Thanks @sergenyalcin, lgtm. I think we have already announced the migration docs in this repository and we will need to sync with the community so that they can find the new place for the documentation after the other docs are published and this is merged.

@jeanduplessis
Copy link
Collaborator

Thanks @sergenyalcin, lgtm. I think we have already announced the migration docs in this repository and we will need to sync with the community so that they can find the new place for the documentation after the other docs are published and this is merged.

@ulucinar the only places we communicated this to the community are:

I can take the action to make the necessary updates to link to the new location.

@sergenyalcin sergenyalcin merged commit 018e640 into crossplane:main Jun 12, 2023
@sergenyalcin sergenyalcin deleted the remove-migrator-docs branch June 12, 2023 14:23
@ytsarev
Copy link
Member

ytsarev commented Jun 13, 2023

Please mention the new location here :)

@jeanduplessis
Copy link
Collaborator

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.

4 participants