Skip to content

Remove the conditional dependency#21

Merged
PhilippMDoerner merged 4 commits into
PhilippMDoerner:masterfrom
ire4ever1190:fix/conditional-dependency
Apr 26, 2026
Merged

Remove the conditional dependency#21
PhilippMDoerner merged 4 commits into
PhilippMDoerner:masterfrom
ire4ever1190:fix/conditional-dependency

Conversation

@ire4ever1190

@ire4ever1190 ire4ever1190 commented Apr 26, 2026

Copy link
Copy Markdown
Contributor

Have been getting this warning in latest devel Nim which leads to the package not getting installed

  Warning:  Declarative parser failed, the file had to be parsed with the VM parser. Please fix your nimble file.
  Warning:  /home/runner/.nimble/pkgs2/lowdb-0.2.1-e26a392547b826d7d3e6566dc0fd76e9f42ead06/lowdb.nimble(9, 2) 'requires' cannot be nested inside control flow statements

Latest Nimble enables the declarative parser by default which is causing lowdb to not be installed correctly since it doesn't like the structure. This removes the conditional and bumps to minimum Nim that db_connector (minimum is 1.7.3 which was a dev version, so made minimum 2.0.0 which was the next stable)

@PhilippMDoerner
PhilippMDoerner self-requested a review April 26, 2026 06:09

@PhilippMDoerner PhilippMDoerner left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the update!

@PhilippMDoerner
PhilippMDoerner merged commit 95b9767 into PhilippMDoerner:master Apr 26, 2026
3 checks passed
@ire4ever1190
ire4ever1190 deleted the fix/conditional-dependency branch April 26, 2026 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants