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

Read filenames using case insensitive extension #2261

Conversation

sumanau7
Copy link
Contributor

@sumanau7 sumanau7 commented Mar 28, 2020

Description

  1. FileSystemSearcher now finds all files ignoring the case of the extension.
  2. Add warning if core config files are found with .yaml extension in the project.

Checklist

  • I have signed the CLA
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

Jacob Beck and others added 30 commits December 4, 2019 14:36
Made a few related fixes to caching behavior w.r.t schemas
Added a test
…igurable-test-warehouses

make tests honor environment variables (dbt-labs#1939)
…ct-downlevel-setuptools

fail more gracefully when setuptools is downlevel (dbt-labs#1975)
…drop-schema

Restore drop_schema to adapter in jinja context (dbt-labs#1980)
…-warning-context

add exceptions.warn to the dbt context (dbt-labs#1970)
Use generated_at instead of datetime.utcnow()
…-materializations

add a formal search order for materializations, tests (dbt-labs#1962)
Clean up retries unit test's connection manager mocking

Co-Authored-By: Jacob Beck <beckjake@users.noreply.github.com>
Jacob Beck and others added 22 commits February 12, 2020 08:13
Add the adapter type to the manifest metadata field.
Pin circle to the newest version of the test container
Have the dockerfile also install pinned versions of things
update dev_requirements.txt to match the dockerfile changes
Add python 3.9 while I am in here
…erfile-pin-stuff

fix a bunch of issues caused by virtualenv 20, tox 3.14, six, and ubuntu (dbt-labs#2128)
…gelog-0160b1

changelog updates, PR template
…ents-update

rev boto3/botocore/snowflake-connector-python/google libraries (dbt-labs#2102)
Correctly handle an empty dbt_project.yml in `dbt debug`
…uery-user-agent

add a user agent with version to the bigquery client info (dbt-labs#2121)
@cla-bot
Copy link

cla-bot bot commented Mar 28, 2020

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Sumanau Sareen.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@sumanau7 sumanau7 changed the title Fix/read filenames using case insenstive extension Read filenames using case insensitive extension Mar 28, 2020
@cla-bot
Copy link

cla-bot bot commented Mar 29, 2020

Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Sumanau Sareen.
This is most likely caused by a git client misconfiguration; please make sure to:

  1. check if your git client is configured with an email to sign commits git config --list | grep email
  2. If not, set it up using git config --global user.email email@example.com
  3. Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails

@sumanau7 sumanau7 force-pushed the fix/read_filenames_using_case_insenstive_extension branch from 989217d to 2f6cb8b Compare March 29, 2020 10:49
@cla-bot cla-bot bot added the cla:yes label Mar 29, 2020
@sumanau7 sumanau7 closed this Mar 29, 2020
@sumanau7 sumanau7 deleted the fix/read_filenames_using_case_insenstive_extension branch March 29, 2020 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.