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

Mix HCPB Materials using fusion-materials-db #26

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

anu1217
Copy link

@anu1217 anu1217 commented Nov 5, 2024

fixes #18, #21

@gonuke
Copy link
Member

gonuke commented Nov 21, 2024

I think that @katielygre has proposed one or two more layers of subdirectory for these examples in #27 - I think it would be good to match her approach.

@anu1217
Copy link
Author

anu1217 commented Nov 22, 2024

I think that @katielygre has proposed one or two more layers of subdirectory for these examples in #27 - I think it would be good to match her approach.

Yes, we decided today to have multiple subdirectories for our examples, but we were a little concerned that making the same subdirectories within separate PRs might lead to a merge conflict. If that's not the case I'll go ahead and match the structure in #27.

@gonuke
Copy link
Member

gonuke commented Nov 22, 2024

Git only really cares about files for merge conflicts, so both making directories should be fine.

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.

Transfer DCLL build dict
2 participants