Skip to content

Conversation

@DaniPopes
Copy link
Member

Motivation

  1. move all crates into crates/
  2. prefix manifest workspace.members with crates/
  3. replace ../testdata/ with ../../testdata

Solution

@DaniPopes DaniPopes force-pushed the crates branch 3 times, most recently from 992b946 to 58bd0be Compare August 11, 2023 01:36
@Evalir Evalir requested review from Evalir and mattsse August 11, 2023 01:55
Copy link
Member

@Evalir Evalir left a comment

Choose a reason for hiding this comment

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

as soon as ci is green this is good imo. let's make the flattening happen

@DaniPopes DaniPopes requested a review from Evalir August 11, 2023 02:42
@Evalir
Copy link
Member

Evalir commented Aug 11, 2023

deferring to @mattsse for final merge

Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

ah yikes, I only saw this one now after merging some other prs...

so there are a few conflicts unfortunately, sorry about that

requires rebase

@DaniPopes DaniPopes force-pushed the crates branch 2 times, most recently from 37b3c13 to ac42114 Compare August 11, 2023 14:59
Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

long overdue,

this will introduce conflicts in open prs, but those should be easily fixable, since this simply moves the files

Copy link
Member

@Evalir Evalir left a comment

Choose a reason for hiding this comment

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

finally!

@mattsse mattsse merged commit 5f22627 into foundry-rs:master Aug 11, 2023
@DaniPopes DaniPopes deleted the crates branch August 11, 2023 18:04
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.

3 participants