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

Fixed bugs, made POSIX compliant, clearer messages #47

Merged
merged 5 commits into from
Dec 10, 2024
Merged

Conversation

mvhulten
Copy link
Contributor

@mvhulten mvhulten commented Dec 6, 2024

This accounts for the complete truth table of (is var set?) and (are we on JSC?). If neither is the case, the scripts aborts.

Furthermore, the abort message format is made consistent with another abort location (where I did not add a suggestion as providing model component parameters is already in the README).

Fixed bug, made mkdir usage POSIX-compliant.

Further, I made messages for "things about to be done" contrast to aborts by making them present continuous tense and adding an ellipsis.

This accounts for the complete truth table of (is var set?) and (are we
on JSC?).  If neither is the case, the scripts aborts.

Furthermore, the abort message format is made consistent with another
abort location (where I did not add a suggestion as providing model
component parameters is already in the README).

Finally, I made messages for "things about to be done" contrast to
aborts by making them present continuous tense and adding an ellipsis.
@mvhulten mvhulten requested a review from kvrigor December 6, 2024 09:09
build_tsmp2.sh Outdated Show resolved Hide resolved
@mvhulten mvhulten changed the title Account for not providing environment on non-JSC machine Fixed bugs, made POSIX compliant, clearer messages Dec 6, 2024
@mvhulten mvhulten marked this pull request as draft December 6, 2024 14:45
@mvhulten mvhulten marked this pull request as ready for review December 6, 2024 15:30
@mvhulten mvhulten merged commit fec8718 into master Dec 10, 2024
1 check passed
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.

2 participants