-
Notifications
You must be signed in to change notification settings - Fork 48
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
Use external grep package in GitHub actions, new site config instructions, and pre-configured tier-1 sites #1419
Conversation
EPIC hosts
|
…eature/find_ext_grep
@AlexanderRichert-NOAA Can you please provide details on the OS grep on Acorn so that we can merge this PR? Thanks. |
This PR is blocking other PRs. Can someone please review? If we don't get the external grep info for acorn before merging, I will create an issue to remind us to add that missing site config update later. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary
External grep in new site configs (documentation), CI, Containers, pre-configured sites (at least tier 1).
Tier 1 (mandatory):
acorn (EMC):deferred to Add external grep to acorn site config #1431Tier 2 (optional):
linux.defaultin documentation (secion "New Site Config")macos.defaultin documentation (secion "New Site Config")Container recipes:
CI:
Testing
See above
Applications affected
None (external
grep
is a prerequisite for the upcoming GitHub CLIgh
package).Systems affected
All.
Dependencies
Issue(s) addressed
Working towards #1105
Checklist