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

[Hotfix] remove post/anl ecf duplicate scripts and link to master instead. #585

Merged

Conversation

aerorahul
Copy link
Contributor

This PR:

  • removes the jgdas_atmos_post_anl.ecf and jgfs_atmos_post_anl.ecf scripts and instead links them to jCDUMP_atmos_post_master.ecf instead as they are copies of the master.
  • updates the script setup_ecflow_links.sh to create links
  • updates .gitignore in the respective post directories to ignore the anl link.

This work is part of #398

This bug was identified during the NCO review by @WeiWei-NCO.

Copy link
Member

@KateFriedman-NOAA KateFriedman-NOAA left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for making this fix @aerorahul !

@KateFriedman-NOAA KateFriedman-NOAA merged commit c34897c into feature/ops-wcoss2 Jan 18, 2022
@KateFriedman-NOAA KateFriedman-NOAA deleted the hotfix/fix-missing-post_anl-resources branch January 18, 2022 18:19
kayeekayee pushed a commit to kayeekayee/global-workflow that referenced this pull request May 30, 2024
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.

3 participants