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 legacy bcs path #634

Merged
merged 1 commit into from
Mar 9, 2023

Conversation

gmao-rreichle
Copy link
Contributor

@gmao-rreichle gmao-rreichle commented Mar 8, 2023

This PR updates the path to the legacy bcs data by pointing to the new "bcs_shared" directory in the GMAO project space on Discover (replacing /discover/nobackup/ltakacs/bcs/).

This is the first step in using (legacy) bcs from the new "bcs_shared" dir. The next and much more complex step will address using bcs in their revised sub-directory structure.

…bcs/ with /discover/nobackup/projects/gmao/bcs_shared/legacy_bcs/
@gmao-rreichle gmao-rreichle changed the base branch from main to develop March 8, 2023 22:04
@github-actions
Copy link

github-actions bot commented Mar 8, 2023

This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged.

1 similar comment
@github-actions
Copy link

github-actions bot commented Mar 8, 2023

This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged.

@biljanaorescanin
Copy link
Contributor

@gmao-rreichle To me these look right. These are only ones I found as well.

@gmao-rreichle gmao-rreichle marked this pull request as ready for review March 9, 2023 17:13
@gmao-rreichle gmao-rreichle requested a review from a team as a code owner March 9, 2023 17:13
@gmao-rreichle
Copy link
Contributor Author

@gmao-rreichle To me these look right. These are only ones I found as well.

Ok, thanks. I suggest you go ahead and approve/merge the PR.

@biljanaorescanin biljanaorescanin merged commit 8864a26 into develop Mar 9, 2023
@gmao-rreichle gmao-rreichle deleted the feature/rreichle/update_legacy_bcs_path branch March 9, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0-diff cleanup documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants