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

Update submodule BLT to v0.3.6 #1316

Merged
merged 3 commits into from
Feb 11, 2021
Merged

Update submodule BLT to v0.3.6 #1316

merged 3 commits into from
Feb 11, 2021

Conversation

TotoGaz
Copy link
Contributor

@TotoGaz TotoGaz commented Feb 10, 2021

No description provided.

@TotoGaz TotoGaz self-assigned this Feb 10, 2021
@TotoGaz TotoGaz added the type: build system Build system issue label Feb 10, 2021
@TotoGaz TotoGaz requested a review from corbett5 February 10, 2021 20:39
@corbett5
Copy link
Contributor

@TotoGaz can you update BLT in LvArray, PAMELA, and PVT too? I know that's three additional PRs but I think it's for the best.

@TotoGaz
Copy link
Contributor Author

TotoGaz commented Feb 10, 2021

Do I have to do it all at once or can I do it with the flow?

@corbett5
Copy link
Contributor

It's honestly probably easiest to do it all at once since each one would require a PR in GEOSX. Add me in each PR and I'll approve them ASAP.

@TotoGaz
Copy link
Contributor Author

TotoGaz commented Feb 11, 2021

@corbett5 can you approve, I'm ready to merge 😝

Copy link
Contributor

@corbett5 corbett5 left a comment

Choose a reason for hiding this comment

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

Sorry for the delay, thanks a bunch!

@TotoGaz TotoGaz merged commit f4e7faf into develop Feb 11, 2021
@TotoGaz TotoGaz deleted the feature/TotoGaz/updateBLT branch February 11, 2021 04:24
corbett5 added a commit that referenced this pull request Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: build system Build system issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants