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

Rename post scripts #1037

Conversation

WalterKolczynski-NOAA
Copy link
Contributor

Description
Rename post scripts to remove the unnecessary "ncep" qualifier. Also changes the executable name for upp to be the one produced by upp (upp.x).

The ush/global_nceppost.sh script is removed as it is not used by any job. We may want to rename the gfs_post.sh script (the one that is used) to global_post.sh to be more reflective of its usage.

Type of change

  • Maintenance (alteration that does not change functionality)

How Has This Been Tested?

  • Forecast-only test on Hera
  • Cycled test on Hera

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • New and existing tests pass with my changes

@WalterKolczynski-NOAA WalterKolczynski-NOAA added the maintenance Regular updates and maintenance work label Sep 29, 2022
@WalterKolczynski-NOAA WalterKolczynski-NOAA self-assigned this Sep 29, 2022
Rename post scripts to remove the unnecessary "ncep" qualifier.
@@ -117,9 +117,9 @@
###############################################################
# Run relevant exglobal script

$SCRgfs/ex${RUN}_atmos_nceppost.sh
$SCRgfs/ex${RUN}_atmos_post.sh

Check warning

Code scanning / shellcheck

SCRgfs is referenced but not assigned.

SCRgfs is referenced but not assigned.
@@ -117,9 +117,9 @@
###############################################################
# Run relevant exglobal script

$SCRgfs/ex${RUN}_atmos_nceppost.sh
$SCRgfs/ex${RUN}_atmos_post.sh

Check warning

Code scanning / shellcheck

Double quote to prevent globbing and word splitting.

Double quote to prevent globbing and word splitting.
@@ -117,9 +117,9 @@
###############################################################
# Run relevant exglobal script

$SCRgfs/ex${RUN}_atmos_nceppost.sh
$SCRgfs/ex${RUN}_atmos_post.sh

Check warning

Code scanning / shellcheck

Prefer putting braces around variable references even when not strictly required.

Prefer putting braces around variable references even when not strictly required.
@@ -117,9 +117,9 @@
###############################################################
# Run relevant exglobal script

$SCRgfs/ex${RUN}_atmos_nceppost.sh
$SCRgfs/ex${RUN}_atmos_post.sh

Check warning

Code scanning / shellcheck

Double quote to prevent globbing and word splitting.

Double quote to prevent globbing and word splitting.

cd $DATA

export POSTGPSH=${POSTGPSH:-$USHgfs/gfs_nceppost.sh}
export POSTGPSH=${POSTGPSH:-$USHgfs/gfs_post.sh}

Check warning

Code scanning / shellcheck

Prefer putting braces around variable references even when not strictly required.

Prefer putting braces around variable references even when not strictly required.
@WalterKolczynski-NOAA
Copy link
Contributor Author

Kate is already making most of these linter fixes in PR #1030, so I'm going to close this PR and hold until that is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Regular updates and maintenance work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant