list comprehension would be redundant

[ArtemIsmagilov]

this version more productive and clearer)

[tobixen]

thank you. I hope to be able to do a new release this week.

[tobixen]

style test seems not to pass now.

[tobixen]

(I do agree that your revision looks nicer than the suggestion from the style test ... but we need those tests to pass)

[tobixen]

(perhaps it's possible to tune the test to be a bit more flexible when it comes to line breaking)

[ArtemIsmagilov]

ok, I want to finish this PR)

[tobixen]

Hmm ... I wonder why github doesn't offer to squash the commits into one commit ... or perhaps there is some button or menu thing I haven't seen. Would you care to run git rebase -i master, let your style commit be a fixup, and force-push?

[ArtemIsmagilov]

@tobixen, look, is squash option, web github

[ArtemIsmagilov]

perhaps you are working through the desktop client github - maybe this functionality is not there

[tobixen]

I should probably turn off some "merge queue"-feature. I had expected that I would be able to approve merges, and then it would be merged after all the test code has succeeded - in reality it merges even with failed tests. Currently the only options I have is "Merge without waiting for requirements to be met (bypass branch protections)", "merge when ready" and "queue and merge in a group".

[ArtemIsmagilov]

Oк, got it

[ArtemIsmagilov]

@tobixen, Can you tell me which part of the file creates these dependencies? Why are they needed during development?

pip freeze
-e git+https://github.com/ArtemIsmagilov/caldav.git@3010eafa1471f3a310205c14f416af48324da18d#egg=caldav
caldav @ file:///home/artem/PycharmProjects/caldav

[ArtemIsmagilov]

looks like it's https://github.com/actions/cache