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

fix(hack/prepare.sh): don't change a URL comment in the Makefile #62

Merged

Conversation

tampakrap
Copy link
Contributor

@tampakrap tampakrap commented Jan 10, 2024

A previous sed command breaks a URL that is in a comment in the Makefile.
This commit reverts that change

Description of your changes

Fixes #

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

@Upbound-CLA
Copy link

Upbound-CLA commented Jan 10, 2024

CLA assistant check
All committers have signed the CLA.

@tampakrap tampakrap marked this pull request as draft January 10, 2024 16:40
@tampakrap tampakrap force-pushed the fix/prepare_comment_templates_url branch from bba0c8d to 55a3cd1 Compare January 25, 2024 19:37
A previous sed command breaks a URL that is in a comment in the
Makefile. This commit reverts that change

Signed-off-by: Theo Chatzimichos <[email protected]>
@tampakrap tampakrap force-pushed the fix/prepare_comment_templates_url branch from 55a3cd1 to 761ec6d Compare January 25, 2024 19:41
@tampakrap tampakrap marked this pull request as ready for review January 27, 2024 21:56
Makefile Outdated Show resolved Hide resolved
@tampakrap tampakrap force-pushed the fix/prepare_comment_templates_url branch from fbc0603 to c94e3b0 Compare April 4, 2024 10:07
@ulucinar ulucinar merged commit 9c6ba93 into crossplane:main Apr 4, 2024
1 check passed
@tampakrap tampakrap deleted the fix/prepare_comment_templates_url branch April 4, 2024 10:56
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