Fork Sync: Update from parent repository #469
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
#3553) * Release 8.7.1 (hotfix) (#3459) * Remove the retention policy setting (#3452) --------- Co-authored-by: Cheick Keita <chkeita@microsoft.com> * Revert "Release 8.7.1 (hotfix) (#3459)" (#3468) This reverts commit c69deed. * Add a ToExpand trait * Implement ToExpand for CommonConfig * Rename to GetExpand and implement for all task configs * Add proptest to the crate and a simple test function (broken) * Fix import issues * Add config tests for analysis task * Implement tests for merge task * Return Result from get_expand() and implement for the rest of the configs * Resolve merge conflict * Undo comment * Update comments * Undo unintentional changes * Add missing newline * Fix build checks * Fix build checks - lifetime elision * Fix linux test imports * Fix indented line * Fix build checks * Replace common test code with a declarative macro * Format code * Simplify URL arbitrary even further to guarantee that it's parseable * Upgrade proptest version * Implement Default for CommonConfig * Remove the GetExpand trait * Return Expand instead of Result<Expand> * cargo fmt * Remove unused import * Fix PR comments (see full message for details) - Stop initializing target_exe in get_expand() for the dotnet coverage task since it's overwritten when the expander is used. - Expand the inputs field from the generic merge task as generated_inputs. * Update dotnet coverage task Expander tests * Remove incorrect expansion variable from generic merge task --------- Co-authored-by: Adam <103067949+AdamL-Microsoft@users.noreply.github.com> Co-authored-by: Cheick Keita <chkeita@microsoft.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.