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

Add poetry dependencies to the codespaces env #367

Closed
wants to merge 8 commits into from

Conversation

shruti222patel
Copy link
Contributor

Why are these changes needed?

Ensures poetry dependencies are installed in the codespaces environment for easier setup.

Related issue number

#343

Checks

@codecov-commenter
Copy link

codecov-commenter commented Oct 22, 2023

Codecov Report

Merging #367 (ab7e4a9) into main (61ac640) will not change coverage.
Report is 9 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #367   +/-   ##
=======================================
  Coverage   42.33%   42.33%           
=======================================
  Files          19       19           
  Lines        2393     2393           
  Branches      539      539           
=======================================
  Hits         1013     1013           
  Misses       1302     1302           
  Partials       78       78           
Flag Coverage Δ
unittests 42.24% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@shruti222patel
Copy link
Contributor Author

DO NOT MERGE -- deleting the .devcontainer/Dockerfile causes the codespace build to break even though .devcontainer/devcontainer.json points to the Dockerfile in the root directory. 🤔

@gagb gagb closed this Aug 26, 2024
jackgerrits added a commit that referenced this pull request Oct 2, 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