-
Notifications
You must be signed in to change notification settings - Fork 107
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
feat: adopt poetry #1172
feat: adopt poetry #1172
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1172 +/- ##
==========================================
- Coverage 71.00% 70.98% -0.02%
==========================================
Files 85 84 -1
Lines 3445 3440 -5
==========================================
- Hits 2446 2442 -4
+ Misses 999 998 -1 ☔ View full report in Codecov by Sentry. |
4f775a8
to
e4af425
Compare
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.
just being picky: for some users using Mac with chip M1 as me, L12 will have to revise to
"export DOCKER_DEFAULT_PLATFORM=linux/arm64/v8"
manually. Maybe note this at quotes inside or readme?
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.
i use M1 and I've never needed to change these before. I believe it's another issue that we need to dig in. Let's open another ticket/issue for this, shall we?
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.
P.S. I followed this instruction to install x86 versions of homebrew and have my terminal opened with rosetta by default, and I also followed this instruction to properly setup terminal profiles in VSCode. I believe these settings have saved me from a bunch of system issues on M1.
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.
good idea. I will move the further discussion at #1175
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.
Thx!
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! successfully worked at my local. I will merge when @josix approve.
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
Types of changes
resolve #1137
Description
Adopt poetry as the package management tool and the virtual env management tool