Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@cesarzc
Copy link
Contributor

@cesarzc cesarzc commented Nov 7, 2020

This change integrates the changes in the feature/performance-tracking-improvements branch to main.

cesarzc and others added 15 commits September 22, 2020 20:40
* Refactor code a bit to be able to make performance tracking more widely available.

* Better perfomance tracking methods.

* It works, clean-up pending.

* Clean-up before publishing PR.

* Apply suggestions from code review

Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>

* Log error when somwthing goes wrong while tracking performance.

* Made two source files nullable enabled.

Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>

Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
…mance-tracking-improvements-201023

Merge 'main' to 'feature/performance-tracking-improvements'
* Expanded compilation tracker interface.

* Added performance tracking tests.

* Fixed project file.

* Created CompilationTrackerTests.fs source file.

* Added a few more test cases in F#.

* Added more test cases to validate time accounting.

* Added expcetion-based test cases.

* Removed Tests.PerformanceTracking C# project.

* Use built-in Assert.Throws.

* Apply suggestions from code review

Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>

* Format type annotations.

Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
…eature-performance-tracking-improvements-to-main'.
@cesarzc cesarzc requested a review from vxfield November 9, 2020 21:28
Copy link
Contributor

@ScottCarda-MS ScottCarda-MS left a comment

Choose a reason for hiding this comment

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

looks good to me

@cesarzc cesarzc merged commit 2327bc9 into main Dec 10, 2020
@cesarzc cesarzc deleted the cesarzc/merge-feature-performance-tracking-improvements-to-main branch May 4, 2021 00:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants