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.

Sign up for GitHub

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

Jump to bottom

🧹 General Cleanup #450

Merged
merged 1 commit into from
Mar 5, 2025
Merged

🧹 General Cleanup #450

merged 1 commit into from
Mar 5, 2025

Conversation

JuaniRios
Copy link
Contributor

@JuaniRios JuaniRios commented Feb 7, 2025 •
edited
Loading

What?

  • Rename files
  • Delete unused functions
  • Fix outdated docs
  • Restructure tests
  • Move runtime api logic to functions folder so we have same structure as tests
  • Fix clippy warnings

This was referenced Feb 7, 2025
Merged
Merged
@JuaniRios Graphite App
Copy link
Contributor Author

JuaniRios commented Feb 7, 2025 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@JuaniRios JuaniRios mentioned this pull request Feb 7, 2025
Merged
@JuaniRios JuaniRios changed the title General Cleanup 🧹 General Cleanup Feb 7, 2025
@JuaniRios JuaniRios force-pushed the 02-07-general_cleanup branch 2 times, most recently from 481a076 to 07e2f57 Compare February 7, 2025 13:07
@JuaniRios JuaniRios self-assigned this Feb 7, 2025
@JuaniRios JuaniRios requested a review from lrazovic February 7, 2025 13:30
@JuaniRios JuaniRios marked this pull request as ready for review February 7, 2025 13:30
@JuaniRios JuaniRios force-pushed the 02-07-general_cleanup branch 2 times, most recently from e515cf3 to c467233 Compare February 11, 2025 13:26
@lrazovic lrazovic force-pushed the 02-07-general_cleanup branch from c467233 to 15d9c00 Compare February 11, 2025 14:37
@lrazovic lrazovic force-pushed the 02-07-general_cleanup branch 4 times, most recently from 9ae610e to a2cfd46 Compare February 11, 2025 16:20
@JuaniRios JuaniRios force-pushed the 02-07-general_cleanup branch from a2cfd46 to caf119f Compare February 12, 2025 12:15
This was referenced Feb 12, 2025
Closed
Closed
Merged
@JuaniRios JuaniRios force-pushed the 02-07-general_cleanup branch 2 times, most recently from 282ae9b to 2d5358e Compare February 13, 2025 16:44
@JuaniRios JuaniRios force-pushed the 02-07-general_cleanup branch from 2d5358e to f048206 Compare February 17, 2025 09:46
@JuaniRios JuaniRios force-pushed the 02-07-general_cleanup branch 2 times, most recently from 061569a to e97182c Compare February 17, 2025 11:18
This was referenced Feb 17, 2025
Merged
Merged
Closed
Merged
Merged
This was referenced Feb 24, 2025
Closed
Merged
@lrazovic lrazovic force-pushed the 02-07-general_cleanup branch from e97182c to acbfda5 Compare February 27, 2025 13:14
@lrazovic lrazovic mentioned this pull request Feb 27, 2025
Merged
This was referenced Mar 3, 2025
Merged
Merged
@lrazovic Graphite App
Copy link
Member

lrazovic commented Mar 5, 2025 •
edited
Loading

Merge activity

  • Mar 5, 4:29 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Mar 5, 4:30 AM EST: Graphite couldn't merge this pull request because a downstack PR 🔥 Funding Rounds Simplification #436 failed to merge.
  • Mar 5, 4:34 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Mar 5, 4:46 AM EST: Graphite rebased this pull request as part of a merge.
  • Mar 5, 4:47 AM EST: A user merged this pull request with Graphite.

@lrazovic lrazovic changed the base branch from 02-06-remove_wap to graphite-base/450 March 5, 2025 09:42
@lrazovic lrazovic changed the base branch from graphite-base/450 to main March 5, 2025 09:44
@lrazovic lrazovic force-pushed the 02-07-general_cleanup branch from acbfda5 to ea242c4 Compare March 5, 2025 09:45
@lrazovic lrazovic merged commit b87a053 into main Mar 5, 2025
1 check passed
@lrazovic lrazovic deleted the 02-07-general_cleanup branch March 5, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lrazovic lrazovic lrazovic approved these changes

Assignees

@JuaniRios JuaniRios

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@JuaniRios @lrazovic