remove class-level sticky session parameter
This cleans up a lot of stuff. Now we can treat the session as a normal feed_manager parameter. Since there is usually only one feed_manager in operation at any time, it is basically a static member....
View Articlemake test suite pass again
This was failing because hooking up the cache into the session completely obliterates our poor old betamax cache. Instead of doing that, we politely queue the cache layer behind it... ... except that...
View Articlerename feeds to feed_manager in main
The "feeds" appelation is an old remnant of the previous data structures. Now we do use a FeedManager everywhere and we should name it as such. It will be easier to grep for it and will more obviously...
View Articlereuse feed_manager object in fetch as well
I see no reason why we need to construct a different object in this specific class, let's just reuse the one already created. This should make a tiny improvement on the startup latency, but hasn't...
View ArticleDRY: refactor common requirements out of tox and setup.py
Requirements are now stored only in setup.py. Before, we were telling tox to delegate the test runs to setup.py, but this made it impossible to pass arguments to pytest. By calling pytest directly, we...
View Articleexplicitely override the feed, not the session
This gives us a single override point that is more cohesive and cleaner. It will also make other run tests much easier to call, but that's for a later commit.
View Articleremove useless --config and --database and override instead
By passing the feed_manager, we have complete and coherent control over those parameters, without having to pass along the arguments. The downside is we create empty cassettes for all of those, but...
View Articleadd empty test cassettes artifacts
Those seem to be created now that we use the betamax session everywhere, but they are probably harmless.
View ArticleREADME: don't let the example config create a folder named '~/Mailbox/'
I was confused that feeds didn't show up in my usualy location, but on further investigation, I found out that I now had a new subfolder in my home folder, literally named '~: $ find ./"'~"...
View ArticleMerge branch 'master' into 'master'
README: don't let the example config create a folder named '~/Mailbox/' See merge request anarcat/feed2exec!7
View Articlerecover from feedparser exceptions (Closes: #964597)
In the aforementionned bug report, feed2exec crashes brutally (with a backtrace, and not completely done) on the following feed: http://www.agendadulibre.org/events.rss?region=12 The full backtrace...
View Articlerun tox on whatever python environment is available
This removes a duplicate source of information about where to store Python versions. The classifiers remain, however. This makes the `tox` command (ran locally on my laptop) work without errors too.
View Articleadd known failing black environment
That .gitlabci came from undertime, so I think it's okay to keep it around. I'm just not ready to basically rewrite the entire source code here just yet however, so silence errors from black for now.
View ArticleFix --loglevel option spec
The --loglevel command line option is not a flag but a choice. Consequently, it needs a *default* and not a *flag_value*.
View ArticleOkay, try a --quiet option instead
As discussed in Issue #13, a way is needed to hide warnings but show errors. Since we apparently cannot have a general loglevel option, this commit provides a way to set that particular loglevel.
View ArticleArgh! Working around click is getting entirely unreasonable.
Looks like the loglevel is _still_ not set unless every option that _may_ set it has a default value. Or?
View ArticleAPI change: allow absolute path in folder setting
This resolves a bunch of issues, detailed in #14. But basically, I was assuming users would never specify an absolute path in `folder`. But it turns out this was natural for at least one user: they...
View Articlesilence pytest warning
the regression mark is some thing I found in another project but I never defined here. it's kind of useless anyways: just use a proper doc string instead.
View Articlemove default database path to ~/.local/share
According to the XDG Base Directory Specification, $XDG_CACHE_HOME is meant to contain "user-specific non-essential (cached) data", and as such that location is often not contained in backups....
View ArticleMerge branch 'for-master' into 'master'
move default database path to `~/.local/share` Closes #16 See merge request anarcat/feed2exec!9
View Articleuse a variable for the data dir as well to clarify the code
This makes the code prettier, at the cost of wasting more time joining strings.
View Articlefix default log level value
The patch in !8 made it so the log level was ERROR by default, which is what the author fundamentally wanted, but it's not actually what the behavior originally was. Restore the previous behavior by...
View Articleuse warnings instead of logging in guess_path
This seems more natural anyways: we want to warn about Deprecation and that's typically the way to do it. We have to revert to some crazy hacking to make it work here, but it works.
View Article