Skip to content
This repository has been archived by the owner on Aug 17, 2024. It is now read-only.

Fixes for cookiejar #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

singularperturbation
Copy link

Was getting compile error when running nimble test, so made the following changes to cookiejar.nim.

  • Timezone now not integer but Timezone object
    • Assume UTC time
  • Strings can't be nil, so check if empty instead

Might be making a few more changes (I was also running into issue #16 due to subexes being removed from the stdlib).

- Timezone now not integer but Timezone object
  - Assume UTC time
- Strings can't be nil, so check if empty instead
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant