Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: remove unnecessary use of floating point #19720

Merged
merged 2 commits into from
Jun 8, 2023

Conversation

benpicco
Copy link
Contributor

@benpicco benpicco commented Jun 8, 2023

Contribution description

Testing procedure

Issues/PRs references

split off #19002

@benpicco benpicco changed the title remove unnecessary use of floating point in tests tests: remove unnecessary use of floating point Jun 8, 2023
@github-actions github-actions bot added the Area: tests Area: tests and testing framework label Jun 8, 2023
@benpicco benpicco added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: tests Area: tests and testing framework and removed Area: tests Area: tests and testing framework labels Jun 8, 2023
@benpicco benpicco requested a review from maribu June 8, 2023 13:54
@benpicco benpicco added the Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer label Jun 8, 2023
@riot-ci
Copy link

riot-ci commented Jun 8, 2023

Murdock results

✔️ PASSED

7634f4c tests/unittests: avoid float in tests-bloom

Artifacts

@benpicco
Copy link
Contributor Author

benpicco commented Jun 8, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Jun 8, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 2268caa into RIOT-OS:master Jun 8, 2023
@benpicco benpicco deleted the float-remove branch June 8, 2023 21:53
@benpicco benpicco added this to the Release 2023.07 milestone Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants