Skip to content

Conversation

@theavege
Copy link
Contributor

No description provided.

@silvioprog
Copy link
Member

@theavege should I merge?

@theavege theavege closed this Jan 14, 2025
@theavege
Copy link
Contributor Author

@silvioprog Would you be comfortable adding console unittests like there?

@theavege theavege reopened this Jan 14, 2025
@silvioprog
Copy link
Member

Hey @theavege, the tests are already included here: https://github.com/risoflora/brookframework/tree/main/Test

Please note that these tests are compatible with both Delphi and Free Pascal.

@theavege
Copy link
Contributor Author

@silvioprog Could you please approve this PR?

@silvioprog silvioprog merged commit c1cede4 into risoflora:main Jan 18, 2025
1 check failed
@silvioprog
Copy link
Member

@theavege approved and merged. Thank you!

@silvioprog
Copy link
Member

@theavege also feel free to join our telegram group here -> https://t.me/brookframework

@theavege theavege deleted the fix/ci branch January 19, 2025 08:57
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