Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Initial playwright setup #1665
Initial playwright setup #1665
Changes from all commits
3601e04
635d99b
304c288
d4f30f7
bd4e939
78e7fc4
481e788
a57f3c4
6d20c98
1f1e141
be6c794
aae1d16
4f8dc3d
ed4e830
6e5ee6d
f6de84d
a2faeee
bac4a59
5b305cd
6ec87f0
2ad22fe
2874c8b
346eaae
dc7b227
8200a4f
5024154
2390753
c755179
70f0f66
c4105a9
50926ab
5f80e59
5f94818
a7ef508
cd39da3
ec249d5
beebd9f
a4b7cba
ccf0f61
a431301
9b101bb
e719bd0
7c6b62a
bee6053
76d78d5
2433510
62ce07b
7da6cc2
726a08d
ca1570b
4886e3f
4da9d4a
8a905a2
f2a1396
c5c9624
7db3696
9965b29
96ba3b1
06bdba7
15a5044
729092a
9e62d31
df1a2ca
93c703b
5245dc4
902b466
74e99c0
9d5ce0b
11d6cf3
7af2be8
2558c4a
efaff5b
76ff6db
a0954b5
eb9a768
e803e2a
e4d2a5f
ece5d19
79aaba0
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Can you explain why we need the whole
dotenv
package? Is this just for debugging locally? If so, can't we manage without an extra dependency? It seems just loading a Python file should suffice?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.
Are you suggesting I do
from dotenv import load_dotenv
?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.
No, the question is rather "Do we need dotenv at all"? Unless I'm missing something, we only use it to set some environment variables that we later use inside the test. Can't we just set them manually? We do we need an extra dependency for that?
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 like it for easy-of-use so that I don't have to set env vars to run a test. I can keep a
.env
in my local repo, have it gitignored, and everything just works. Otherwise, I'd have to manually set them in the py file before I ran locally. Which would then cause a git diff. I added it asdev
requirement, which I didn't think would be that big a deal. If you take serious issue with it, I can remove.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 personally like to keep the needed tools / requirements to a minimum to keep the mental load small. That being said, without anyone else chiming in, this is a "you vs. me" situation. Unless we want to have a more general discussion about this with the team now, I'm ok to withdraw my concern in favor of velocity.