Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

svrooij

svrooij created a comment on a pull request on microsoft/kiota-python
![image](https://github.com/user-attachments/assets/0b56cee8-26ea-42f0-a991-5b3cc26223ee) ๐Ÿ‘€

View on GitHub

svrooij created a comment on an issue on microsoft/kiota-python
@baywet maybe we can do this for version 1.8.0 as well?

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
It really helps that I'm now able to run all the tests locally in Python 3.9, only 1 error left and that is because an assertion tried to parse a datetime string with a z to compare

View on GitHub

svrooij pushed 1 commit to remove-pendulum-dependency svrooij/kiota-python
  • Fixing datetime parsing in test ๐Ÿงช f1b515f

View on GitHub

svrooij pushed 1 commit to remove-pendulum-dependency svrooij/kiota-python
  • More ๐Ÿ•‘ compatibility 1373232

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
@danieleds and strip of the `Z`? ``` FAILED tests/unit/test_json_parse_node.py::test_get_collection_of_object_values - ValueError: Invalid isoformat string: '2021-07-29T03:07:25Z' ```

View on GitHub

svrooij pushed 1 commit to remove-pendulum-dependency svrooij/kiota-python
  • Hopefully try 50 fixes it ๐Ÿ’ฃ 15b5a31

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
Going to download a python 3.9 dev container and run the tests there.

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
@baywet I only get notifications from reactions (yours and sonarqube)

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
> Oh I see. Yes, that's exactly what I had in mind. Something like, in form_parse_node.py and similar files: > > ```python > if sys.version_info[:3] <= (3,10): > fixed_time = re.sub(r'(\d[...

View on GitHub

svrooij pushed 1 commit to remove-pendulum-dependency svrooij/kiota-python
  • fromisoformat compatibility 8aaeff1

View on GitHub

svrooij created a review comment on a pull request on microsoft/kiota-python
Yes ?P is python regex for, this is a named group

View on GitHub

svrooij created a review on a pull request on microsoft/kiota-python

View on GitHub

svrooij pushed 1 commit to remove-pendulum-dependency svrooij/kiota-python

View on GitHub

svrooij created a comment on an issue on microsoft/kiota-python
This is what I meant... I have no clue about python but this documentation would be useful. Created a devcontainer config that does this and allows your to run all those things from vscode, no int...

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
> > To make it not stop working my suggestion is that I make the P optional. > > Ah no, good catch, it's a typo on my end, please add the P, it's mandatory according to the ABNF. Want me to r...

View on GitHub

svrooij pushed 1 commit to remove-pendulum-dependency svrooij/kiota-python
  • Support additional timedeltas f0ed780

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
Got the tests working in a dev container. Down to the last issue. ```plain text = 'T12H59M45S' def parse_timedelta_from_iso_format(text: str) -> timedelta: """Parses a ISO8601 d...

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
@baywet I'll try to get it over the finish line, tomorrow.

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
This is related to #453 being able to run the tests locally would greatly reduce the developer feedback flow.

View on GitHub

svrooij pushed 1 commit to remove-pendulum-dependency svrooij/kiota-python
  • Reduce complexity and raise error instead of return none e61eabb

View on GitHub

svrooij pushed 2 commits to remove-pendulum-dependency svrooij/kiota-python
  • Fix issues caused by auto format 74fbeeb
  • More pendulem removed e6eb346

View on GitHub

svrooij created a comment on a pull request on microsoft/kiota-python
@baywet is there a command to apply the wanted code formatting? I see it's failing at that.

View on GitHub

svrooij pushed 1 commit to remove-pendulum-dependency svrooij/kiota-python

View on GitHub

svrooij opened an issue on microsoft/kiota-python
Lower requirement for urllib and httpx
Are the dependency requirements set in stone? Or can we lower `urllib` and `httpx` to something that works with these package constrains? https://github.com/microsoft/kiota-python/blob/dbda749947f...
svrooij pushed 3 commits to remove-pendulum-dependency svrooij/kiota-python
  • fix: bumps pendulum to a stable version Signed-off-by: Vincent Biret <[email protected]> 75b302f
  • Merge pull request #449 from microsoft/chore/pendulum-stable fix: bumps pendulum to a stable version dbda749
  • Merge branch 'main' into remove-pendulum-dependency f2ad78e

View on GitHub

svrooij pushed 1 commit to remove-pendulum-dependency svrooij/kiota-python
  • Feedback processed and removed python-dateutil 41d8efd

View on GitHub

svrooij created a review comment on a pull request on microsoft/kiota-python
Added

View on GitHub

svrooij created a review on a pull request on microsoft/kiota-python

View on GitHub

svrooij created a review comment on a pull request on microsoft/kiota-python
> The timedelta class in Python's datetime module does not enforce faulty combinations like months and weeks. Instead, it computes the expected value based on the provided arguments. The timedelta ...

View on GitHub

Load more