Malthe Jørgensen

Results 11 comments of Malthe Jørgensen

Reading the code it seems that you should be able use the `id` keyword argument to set the job id, e.g.: ``` scheduler.schedule(date, function, args=args, kwargs=kwargs, id=job_id) ```

@Radiergummi Nice – that will work for many use-cases. Just note that it doesn't work when the individual records contain newlines – e.g. user-generated content like posts in a StackOverflow...

They are escaped by putting `"` around the value, so the raw newlines are still present in the csv, meaning that `uniq` will not detect duplicates that contain newlines ```...

For what it's worth, our (@tony and I) issue was this: https://github.com/graphql-python/graphql-core-legacy/pull/287

This change does move the cursor to a different place than where it ended before, which can be seen in the tests.

For reference, this is what I get locally when running `UnitTesting` on NeoVintageous. ``` reloading plugin NeoVintageous.tests.commands tests (unittest.loader._FailedTest) ... ERROR ====================================================================== ERROR: tests (unittest.loader._FailedTest) ---------------------------------------------------------------------- ImportError: Failed to import...

Hi @dsilhavy. The PR is now in a reviewable state, but not finalized since I'd like someone more experienced with this codebase than me (that person is probably you 😁)...

@jairhenrique I've rebased this branch as well as updated the base branch from `master` to `main`. The code is quite old so I'm not 100% whether it works with the...

Tests / Python 3.11 (pull_request) was failing due to linting/`black` so I updated the commit to follow `black` formatting as set up in this repo (line length: 90).

Just noting B905 has the same issue for `zip`. If many people define their own `groupby` I wonder if the rule should be off by default 🤷‍♂️