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

Use absolute path for project_dir context object #414

Closed
wants to merge 1 commit into from

Conversation

Cbeck527
Copy link

The --project-dir option was not correctly calculating the path to open up the
config file. This would result in valid project_dir values raising an OSError
and presenting this error message to the user:

$ ls
drwxr-xr-x  6 chris  staff   204B Jul 12 17:24 sample/
...
...
$ chalice --project-dir sample deploy
Unable to load the project config file. Are you sure this is a chalice project?

The --project-dir option was not correctly calculating the path to open up the
config file. This would result in valid `project_dir` values raising an OSError
and presenting this error message to the user:

```
$ ls
drwxr-xr-x  6 chris  staff   204B Jul 12 17:24 sample/
...
...
$ chalice --project-dir sample deploy
Unable to load the project config file. Are you sure this is a chalice project?
```
@codecov-io
Copy link

Codecov Report

Merging #414 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #414      +/-   ##
==========================================
+ Coverage   93.54%   93.58%   +0.03%     
==========================================
  Files          18       18              
  Lines        2805     2804       -1     
  Branches      369      368       -1     
==========================================
  Hits         2624     2624              
  Misses        133      133              
+ Partials       48       47       -1
Impacted Files Coverage Δ
chalice/cli/__init__.py 78.21% <100%> (+0.43%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c458a5c...9c805f4. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants