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: Update cat-gateway rust build, cleaunup markdowns, code and earthly files #87

Closed
wants to merge 7 commits into from

Conversation

Mr-Leshiy
Copy link
Contributor

Description

  • Updated cat-gateway rust build, remove using cargo chef.
  • Bump rust toolchain version to 1.73.0
  • Cleanup earthly files and markdowns

Related Pull Requests

input-output-hk/catalyst-ci#54

@Mr-Leshiy Mr-Leshiy self-assigned this Oct 25, 2023
@minikin minikin added the review me PR is ready for review label Oct 25, 2023
@minikin minikin enabled auto-merge (squash) October 25, 2023 12:34
@stevenj
Copy link
Collaborator

stevenj commented Nov 1, 2023

@Mr-Leshiy I think this is out of date now with respect to the Rust changes I ended up making.
Can you check it?

Copy link
Collaborator

@stevenj stevenj left a comment

Choose a reason for hiding this comment

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

Please rebase this PR on the latest cat-gateway code, some of these changes are already incorporated in the main and may not be 100% exactly as in the PR.

@Mr-Leshiy
Copy link
Contributor Author

@stevenj yep, sure, checking it.
With deeply in stepping PostgreSQL stuff, missed your comments

@Mr-Leshiy
Copy link
Contributor Author

@stevenj closing this PR because it is not relevant anymore.
For minor fix I have created a new one #114

@Mr-Leshiy Mr-Leshiy closed this Nov 3, 2023
auto-merge was automatically disabled November 3, 2023 09:15

Pull request was closed

@Mr-Leshiy Mr-Leshiy deleted the fix/event-db-readme branch November 3, 2023 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review me PR is ready for review
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants