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

refactor: move [Dune_load.Dune_files.in_dir] #9768

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

rgrinberg
Copy link
Member

It doesn't need to be in the [Dune_files] submodule

Signed-off-by: Rudi Grinberg me@rgrinberg.com

It doesn't need to be in the [Dune_files] submodule

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>

<!-- ps-id: 63511604-57a0-40fa-93f9-0259abbeaf65 -->
@rgrinberg rgrinberg merged commit 01ee7b7 into main Jan 18, 2024
26 of 27 checks passed
@rgrinberg rgrinberg deleted the ps/rr/refactor__move__dune_load_dune_files_in_dir_ branch January 18, 2024 04:08
emillon pushed a commit to emillon/dune that referenced this pull request Feb 5, 2024
It doesn't need to be in the [Dune_files] submodule

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
@emillon emillon mentioned this pull request Feb 5, 2024
emillon added a commit that referenced this pull request Feb 5, 2024
* refactor: rename [Dune_load.conf] to [Dune_load.t] (#9766)

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>

* refactor: Make [Dune_load.t] abstract (#9767)

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>

* refactor: move [Dune_load.Dune_files.in_dir] (#9768)

It doesn't need to be in the [Dune_files] submodule

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>

* fix: performance regression from #8447 (#9769)

Get rid of the slow dune file comparison in #9738

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>

---------

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Co-authored-by: Rudi Grinberg <me@rgrinberg.com>
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.

1 participant