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

Complete the attachment processors extension #3161

Merged
merged 47 commits into from
Jan 12, 2022

Conversation

MarcoRossignoli
Copy link
Contributor

@MarcoRossignoli MarcoRossignoli commented Nov 9, 2021

Complete attachment processors extension

fixes #2379

Missing part unrelated to this repo:

  • Update documentation for attachment processor discovery
  • Move coverage attachment processor to coverage repo
  • VS integration

contributes to #1811

@MarcoRossignoli
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Pull request contains merge conflicts.

@MarcoRossignoli
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MarcoRossignoli MarcoRossignoli force-pushed the mergeattachments branch 2 times, most recently from d9dce9c to 05f6157 Compare November 20, 2021 21:58
@MarcoRossignoli MarcoRossignoli changed the title Attachment processors - exercise CI Complete attachment processors extension Nov 22, 2021
@MarcoRossignoli MarcoRossignoli marked this pull request as ready for review November 23, 2021 18:10
@MarcoRossignoli MarcoRossignoli enabled auto-merge (squash) November 23, 2021 18:11
@MarcoRossignoli MarcoRossignoli enabled auto-merge (squash) November 23, 2021 19:07
@MarcoRossignoli MarcoRossignoli marked this pull request as draft November 24, 2021 10:31
@MarcoRossignoli
Copy link
Contributor Author

/azp run

@MarcoRossignoli MarcoRossignoli merged commit 1c9f0fb into microsoft:main Jan 12, 2022
@MarcoRossignoli MarcoRossignoli deleted the mergeattachments branch January 12, 2022 09:58
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.

Possible confusing logging on ProxyOperationManager.cs
5 participants