Skip to content

Conversation

@limingchina
Copy link
Contributor

The current function name is quite generic and it's easy to conflict
with other functions in a project since they're C functions

The current function name is quite generic and it's easy to conflict
with other functions in a project since they're C functions
@kevswims
Copy link

The now() function also conflicts with other libraries (libsystemd in our case). Can renaming that be added to this PR?

@limingchina
Copy link
Contributor Author

The now() function also conflicts with other libraries (libsystemd in our case). Can renaming that be added to this PR?

Renamed. However, not sure if the author is actively accepting PRs and merge that to the mainline.

@yse
Copy link
Owner

yse commented Aug 28, 2024

Hi! There are a few changes that I should merge and make new major release. The fix is good to up a major version, also migrate to Qt6. Thank you for your contribution. I think I will find time and energy to make project more freshy)

@yse yse merged commit 5e85aea into yse:develop Sep 20, 2024
@yse
Copy link
Owner

yse commented Sep 20, 2024

Great work 🏆 Thank you

I’ll be updating the major version to 3 to stay aligned with semantic versioning.

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.

3 participants