Skip to content
This repository has been archived by the owner on Feb 2, 2021. It is now read-only.

Make Grep case insensitive in download script #195

Merged
merged 1 commit into from
Feb 28, 2020

Conversation

Waterdrips
Copy link
Contributor

##Description

Github made the location header lowercase, so our get.sh script was
failing. This makes the grep case inseneitive.

How Has This Been Tested?

Tested by running the script to make sure it failed, then making the
grep into a grep -i and running again, this time it worked.

Checklist:

I have:

  • checked my changes follow the style of the existing code / OpenFaaS repos
  • updated the documentation and/or roadmap in README.md
  • read the CONTRIBUTION guide
  • signed-off my commits with git commit -s
  • added unit tests

Github made the location header lowercase, so our get.sh script was
failing. This makes the grep case inseneitive.

Tested by running the script to make sure it failed, then making the
grep into a `grep -i` and running again, this time it worked.

Signed-off-by: Alistair Hey <[email protected]>
Copy link
Member

@alexellis alexellis left a comment

Choose a reason for hiding this comment

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

Lgtm

@alexellis alexellis merged commit ee11754 into openfaas:master Feb 28, 2020
@Waterdrips Waterdrips deleted the make-grep-case-insensitive branch June 2, 2020 10:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants