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.

Sign up for GitHub

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

Jump to bottom

Parse unitPreferencesTest.txt preparing to run data-driven tests #26

Merged
merged 7 commits into from Mar 17, 2020
Merged

Parse unitPreferencesTest.txt preparing to run data-driven tests #26

merged 7 commits into from Mar 17, 2020

Conversation

hugovdm
Copy link
Collaborator

@hugovdm hugovdm commented Mar 12, 2020 •
edited

No description provided.

@hugovdm
Copy link
Collaborator Author

hugovdm commented Mar 12, 2020

This diff will be smaller once #25 has been merged - please consider reviewing and merging that first?

@hugovdm
Copy link
Collaborator Author

hugovdm commented Mar 13, 2020

I wasn't sure how to make the diff be against the updated units-staging branch (to pick up PR #25), so I went ahead and rebased it on that branch.

@hugovdm hugovdm requested a review from sffc March 13, 2020 13:16
sffc
sffc reviewed Mar 15, 2020
View reviewed changes
Copy link
Owner

@sffc sffc left a comment

Choose a reason for hiding this comment

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

Mostly LGTM, just some small things

icu4c/source/test/intltest/unitstest.cpp Outdated Show resolved Hide resolved
icu4c/source/test/intltest/unitstest.cpp Outdated Show resolved Hide resolved
icu4c/source/test/intltest/unitstest.cpp Show resolved Hide resolved
icu4c/source/test/intltest/unitstest.cpp Show resolved Hide resolved
@hugovdm
Copy link
Collaborator Author

hugovdm commented Mar 16, 2020

Exploring good ways to handle github code reviews: this time I'm not clicking on "resolved" - I'm leaving that to you. I think they're all resolved though, and I might as well have clicked them myself.

@sffc sffc merged commit 9e021f9 into sffc:units-staging Mar 17, 2020
@sffc
Copy link
Owner

sffc commented Mar 17, 2020

Exploring good ways to handle github code reviews: this time I'm not clicking on "resolved" - I'm leaving that to you. I think they're all resolved though, and I might as well have clicked them myself.

Yep, sounds good, I'll click resolve to verify that they are resolved.

@hugovdm hugovdm mentioned this pull request Apr 3, 2020
Merged
5 tasks
hugovdm added a commit to hugovdm/icu that referenced this pull request Jul 15, 2020
@hugovdm
unitsdata.cpp and associated tests
e968d88 
PR: sffc#20
Commit: 9f65f41
ICU-20568 Read unitsTest.txt, prepare to run unit tests.

PR: sffc#25
Commit: 4be0a2b
No-op/cosmetic: reformat with clang-format ident:4 line-length:105.

PR: sffc#26
Commit: 9e021f9
Parse unitPreferencesTest.txt preparing to run data-driven tests

PR: sffc#34
Commit: 12a5289
Improve test output: no more fprintfs.

PR: sffc#33
Commit: 44e9afb
Update to latest CLDR test files: b7a23f3f41 CLDR-13587.

PR: sffc#31
Commit: ca34233
Add unitsdata.cpp, getConversionRatesInfo, and unit tests.

PR: sffc#41
Commit: be7f699
Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master
hugovdm added a commit to hugovdm/icu that referenced this pull request Jul 23, 2020
@hugovdm
unitsdata.cpp and associated tests
33c6e99 
PR: sffc#20
Commit: 9f65f41
ICU-20568 Read unitsTest.txt, prepare to run unit tests.

PR: sffc#25
Commit: 4be0a2b
No-op/cosmetic: reformat with clang-format ident:4 line-length:105.

PR: sffc#26
Commit: 9e021f9
Parse unitPreferencesTest.txt preparing to run data-driven tests

PR: sffc#34
Commit: 12a5289
Improve test output: no more fprintfs.

PR: sffc#33
Commit: 44e9afb
Update to latest CLDR test files: b7a23f3f41 CLDR-13587.

PR: sffc#31
Commit: ca34233
Add unitsdata.cpp, getConversionRatesInfo, and unit tests.

PR: sffc#41
Commit: be7f699
Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master
hugovdm added a commit to hugovdm/icu that referenced this pull request Aug 3, 2020
@hugovdm
unitsdata.cpp and associated tests
a6a91ca 
PR: sffc#20
Commit: 9f65f41
ICU-20568 Read unitsTest.txt, prepare to run unit tests.

PR: sffc#25
Commit: 4be0a2b
No-op/cosmetic: reformat with clang-format ident:4 line-length:105.

PR: sffc#26
Commit: 9e021f9
Parse unitPreferencesTest.txt preparing to run data-driven tests

PR: sffc#34
Commit: 12a5289
Improve test output: no more fprintfs.

PR: sffc#33
Commit: 44e9afb
Update to latest CLDR test files: b7a23f3f41 CLDR-13587.

PR: sffc#31
Commit: ca34233
Add unitsdata.cpp, getConversionRatesInfo, and unit tests.

PR: sffc#41
Commit: be7f699
Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master
hugovdm added a commit to hugovdm/icu that referenced this pull request Aug 3, 2020
@hugovdm
unitsdata.cpp and associated tests (bundle of PRs)
1329a29 
PR: sffc#20
Commit: 9f65f41
ICU-20568 Read unitsTest.txt, prepare to run unit tests.

PR: sffc#25
Commit: 4be0a2b
No-op/cosmetic: reformat with clang-format ident:4 line-length:105.

PR: sffc#26
Commit: 9e021f9
Parse unitPreferencesTest.txt preparing to run data-driven tests

PR: sffc#34
Commit: 12a5289
Improve test output: no more fprintfs.

PR: sffc#33
Commit: 44e9afb
Update to latest CLDR test files: b7a23f3f41 CLDR-13587.

PR: sffc#31
Commit: ca34233
Add unitsdata.cpp, getConversionRatesInfo, and unit tests.

PR: sffc#41
Commit: be7f699
Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master
hugovdm added a commit to hugovdm/icu that referenced this pull request Aug 15, 2020
@hugovdm
unitsdata.cpp and associated tests (bundle of PRs)
3f84b1b 
PR: sffc#20
Commit: 9f65f41
ICU-20568 Read unitsTest.txt, prepare to run unit tests.

PR: sffc#25
Commit: 4be0a2b
No-op/cosmetic: reformat with clang-format ident:4 line-length:105.

PR: sffc#26
Commit: 9e021f9
Parse unitPreferencesTest.txt preparing to run data-driven tests

PR: sffc#34
Commit: 12a5289
Improve test output: no more fprintfs.

PR: sffc#33
Commit: 44e9afb
Update to latest CLDR test files: b7a23f3f41 CLDR-13587.

PR: sffc#31
Commit: ca34233
Add unitsdata.cpp, getConversionRatesInfo, and unit tests.

PR: sffc#41
Commit: be7f699
Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master
hugovdm added a commit to hugovdm/icu that referenced this pull request Aug 28, 2020
@hugovdm
unitsdata.cpp and associated tests (bundle of PRs)
c2c3af6 
PR: sffc#20
Commit: 9f65f41
ICU-20568 Read unitsTest.txt, prepare to run unit tests.

PR: sffc#25
Commit: 4be0a2b
No-op/cosmetic: reformat with clang-format ident:4 line-length:105.

PR: sffc#26
Commit: 9e021f9
Parse unitPreferencesTest.txt preparing to run data-driven tests

PR: sffc#34
Commit: 12a5289
Improve test output: no more fprintfs.

PR: sffc#33
Commit: 44e9afb
Update to latest CLDR test files: b7a23f3f41 CLDR-13587.

PR: sffc#31
Commit: ca34233
Add unitsdata.cpp, getConversionRatesInfo, and unit tests.

PR: sffc#41
Commit: be7f699
Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master
hugovdm added a commit to hugovdm/icu that referenced this pull request Sep 1, 2020
@hugovdm
ICU-20568 unitsdata.cpp and associated tests (7 PRs)
15da9df 
PR: sffc#20
Commit: 9f65f41
ICU-20568 Read unitsTest.txt, prepare to run unit tests.

PR: sffc#25
Commit: 4be0a2b
No-op/cosmetic: reformat with clang-format ident:4 line-length:105.

PR: sffc#26
Commit: 9e021f9
Parse unitPreferencesTest.txt preparing to run data-driven tests

PR: sffc#34
Commit: 12a5289
Improve test output: no more fprintfs.

PR: sffc#33
Commit: 44e9afb
Update to latest CLDR test files: b7a23f3f41 CLDR-13587.

PR: sffc#31
Commit: ca34233
Add unitsdata.cpp, getConversionRatesInfo, and unit tests.

PR: sffc#41
Commit: be7f699
Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master
hugovdm added a commit to hugovdm/icu that referenced this pull request Sep 9, 2020
@hugovdm
ICU-20568 unitsdata.cpp/.h and associated tests
f4d6251 
Read unitsTest.txt, prepare to run unit tests.
PR: sffc#20
Commit: 9f65f41

No-op/cosmetic: reformat with clang-format ident:4 line-length:105.
PR: sffc#25
Commit: 4be0a2b

Parse unitPreferencesTest.txt preparing to run data-driven tests
PR: sffc#26
Commit: 9e021f9

Improve test output: no more fprintfs.
PR: sffc#34
Commit: 12a5289

Update to latest CLDR test files: b7a23f3f41 CLDR-13587.
PR: sffc#33
Commit: 44e9afb

Add unitsdata.cpp, getConversionRatesInfo, and unit tests.
PR: sffc#31
Commit: ca34233

Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master
PR: sffc#41
Commit: be7f699
hugovdm added a commit to unicode-org/icu that referenced this pull request Sep 10, 2020
@hugovdm
ICU-20568 unitsdata.cpp/.h and associated tests
cf46b41 
Read unitsTest.txt, prepare to run unit tests.
PR: sffc#20
Commit: 9f65f41

No-op/cosmetic: reformat with clang-format ident:4 line-length:105.
PR: sffc#25
Commit: 4be0a2b

Parse unitPreferencesTest.txt preparing to run data-driven tests
PR: sffc#26
Commit: 9e021f9

Improve test output: no more fprintfs.
PR: sffc#34
Commit: 12a5289

Update to latest CLDR test files: b7a23f3f41 CLDR-13587.
PR: sffc#33
Commit: 44e9afb

Add unitsdata.cpp, getConversionRatesInfo, and unit tests.
PR: sffc#31
Commit: ca34233

Update unitsTest.txt and unitPreferencesTest.txt from current CLDR master
PR: sffc#41
Commit: be7f699
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sffc sffc sffc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@hugovdm @sffc