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

Document using xcstrings-tool cli directly #103

Merged
merged 1 commit into from
Jul 1, 2024
Merged

Conversation

liamnichols
Copy link
Owner

this change adds documentation for using the cli directly in advanced use cases (or when SPM plugins are bugged)

Copy link

✅ Pull Request passed benchmark checks

Summary

Sat Jun 29 08:22:38 UTC 2024

===========================================================================================================
Threshold deviations for StringGenerator.generateSource(for:tableName:accessLevel:):XCStringsToolBenchmarks
===========================================================================================================
Time (total CPU) (ms, %) main pull_request Difference % Threshold %
p25 982 1047 6 5
p50 982 1047 6 5

New baseline 'pull_request' is WORSE than the 'main' baseline thresholds.


Comparing results between 'main' and 'pull_request'

Host 'Mac-1719647334730.local' with 3 'arm64' processors with 7 GB memory, running:
Darwin Kernel Version 23.5.0: Wed May  1 20:12:39 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_VMAPPLE

XCStringsToolBenchmarks

StringGenerator.generateSource(for:tableName:accessLevel:) metrics

Time (total CPU) (ms) * p0 p25 p50 p75 p90 p99 p100 Samples
main 982 982 982 1026 1026 1026 1026 2
pull_request 1047 1047 1047 1047 1047 1047 1047 1
Δ 65 65 65 21 21 21 21 -1
Improvement % -7 -7 -7 -2 -2 -2 -2 -1

@liamnichols liamnichols marked this pull request as ready for review June 29, 2024 08:48
@liamnichols liamnichols merged commit ca65df3 into main Jul 1, 2024
13 checks passed
@liamnichols liamnichols deleted the ln/cli-documentation branch July 1, 2024 04:09
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.

1 participant