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

Old commit disapeared.... Reapplied patch #53

Merged
merged 2 commits into from
Sep 12, 2022

Conversation

hulto
Copy link
Collaborator

@hulto hulto commented Sep 12, 2022

What type of PR is this?

/kind bug

What this PR does / why we need it:

Eldritch CI is failing. Git commit for private starlark disappeared recreated PRing.
image

https://github.com/hulto/starlark-rust/tree/v0.6.0-p1

@hulto hulto requested a review from Cictrone September 12, 2022 03:51
Copy link
Collaborator

@Cictrone Cictrone 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 sus af, maybe we should use a tag or something?

@hulto
Copy link
Collaborator Author

hulto commented Sep 12, 2022

This is sus af, maybe we should use a tag or something?

Sure, I think a branch is more readable but sus cuz it can be updated without us updating the realm code base.
I think it'll be okay since I'm the only one that would push to it but not ideal.

@hulto hulto merged commit 3a70033 into main Sep 12, 2022
@hulto hulto deleted the resetting-git-rev-cargo-starlark branch September 12, 2022 23:33
KCarretto pushed a commit that referenced this pull request Feb 1, 2024
 
Old commit disapeared.... Reapplied patch (#53)

* Old commit disapeared.... Reapplied patch as branch
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.

2 participants