Skip to content

Conversation

@nyqykk
Copy link
Member

@nyqykk nyqykk commented Nov 4, 2024

Summary

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@nyqykk nyqykk enabled auto-merge (squash) November 4, 2024 08:27
Copy link
Member

@chenjiahan chenjiahan left a comment

Choose a reason for hiding this comment

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

We should be careful about exporting public APIs, and all public APIs should be documented later.

For private APIs, use prefixes like _ or _internal.

@codspeed-hq
Copy link

codspeed-hq bot commented Nov 4, 2024

CodSpeed Performance Report

Merging #367 will not alter performance

Comparing chore/export-utils (3e06d99) with main (d5d5bb5)

Summary

✅ 5 untouched benchmarks

@nyqykk nyqykk merged commit f23ae2d into main Nov 4, 2024
20 checks passed
@nyqykk nyqykk deleted the chore/export-utils branch November 4, 2024 09:00
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.

4 participants