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

File tree stuff #3679

Merged
merged 20 commits into from
Aug 28, 2024
Merged

File tree stuff #3679

merged 20 commits into from
Aug 28, 2024

Conversation

lf94
Copy link
Contributor

@lf94 lf94 commented Aug 27, 2024

Resolves #3629

Replaces #3629 because the branch name is actually breaking CI

Copy link

qa-wolf bot commented Aug 27, 2024

QA Wolf here! As you write new code it's important that your test coverage is keeping up.
Click here to request test coverage for this PR!

Copy link

vercel bot commented Aug 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
modeling-app ✅ Ready (Inspect) Visit Preview Aug 28, 2024 0:44am

@Irev-Dev
Copy link
Collaborator

Does 2f50804 look okay to you @lf94 ?

@lf94 lf94 merged commit 5e8227e into main Aug 28, 2024
24 checks passed
@lf94 lf94 deleted the file-tree-stuff branch August 28, 2024 10:38
lf94 added a commit that referenced this pull request Aug 28, 2024
* Fix and test file tree operations

* fmt

Signed-off-by: Jess Frazelle <github@jessfraz.com>

* Make tsc happy

* I've been lied to

* Fix navigating to deleted file

* tsc

* Remove debugger statement

* Fix test

* All tests fixed

* Remove old config and remove slowmo

* fmt

* Remove unintentional changelog in readme (#3678)

* lint

* fmt

* Increase test timeout

* Fix the damn test

* fix web app

* fmt

---------

Signed-off-by: Jess Frazelle <github@jessfraz.com>
Co-authored-by: Jess Frazelle <github@jessfraz.com>
Co-authored-by: Jonathan Tran <jonnytran@gmail.com>
Co-authored-by: Kurt Hutten Irev-Dev <k.hutten@protonmail.ch>
@pierremtb pierremtb mentioned this pull request Sep 3, 2024
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