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

Rename local/broker transformers for clarity #577

Merged
merged 1 commit into from
Apr 7, 2023

Conversation

skitt
Copy link
Member

@skitt skitt commented Mar 24, 2023

This makes the direction of transformers and syncers explicit. The documentation is updated to match.

This will require matching changes in Submariner and Lighthouse.

Depends on submariner-io/releases#644

@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr577/skitt/rename-local-broker

@github-advanced-security
Copy link

You have successfully added a new Grype configuration .github/workflows/linting.yml:vulnerability-scan. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

Copy link
Contributor

@tpantelis tpantelis left a comment

Choose a reason for hiding this comment

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

This is an API change - let's hold off until after RC0.

@github-actions
Copy link

github-actions bot commented Apr 3, 2023

This PR/issue depends on:

@dfarrell07
Copy link
Member

The release-0.15 branch is cut, so this can go into devel.

This makes the direction of transformers and syncers explicit. The
documentation is updated to match.

Signed-off-by: Stephen Kitt <[email protected]>
@tpantelis tpantelis merged commit a578003 into submariner-io:devel Apr 7, 2023
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr577/skitt/rename-local-broker]

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