Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Implement the containers function for C# #2078

Merged
merged 8 commits into from
Jul 12, 2022
Merged

Implement the containers function for C# #2078

merged 8 commits into from
Jul 12, 2022

Conversation

Porges
Copy link
Member

@Porges Porges commented Jun 23, 2022

Summary of the Pull Request

Fairly straightforward but required implementation of a few more functions on the IContainers/Containers class.

PR Checklist

  • Applies to work item: Convert containers function to C# #2077
  • CLA signed. If not, go over here and sign the CLI.
  • Tests added/passed
  • Requires documentation to be updated
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

@Porges Porges force-pushed the function/containers branch from f181c05 to 5243738 Compare June 23, 2022 03:10
@Porges Porges self-assigned this Jun 23, 2022
Base automatically changed from function/download to main June 23, 2022 03:48
@Porges Porges force-pushed the function/containers branch from d3e39c7 to 18c92cf Compare June 23, 2022 03:49
@Porges Porges force-pushed the function/containers branch 2 times, most recently from 70eea31 to 083db3d Compare June 30, 2022 04:25
@Porges Porges enabled auto-merge (squash) July 5, 2022 20:15
@Porges Porges force-pushed the function/containers branch 2 times, most recently from 18969a6 to 61b5bb7 Compare July 12, 2022 20:15
@Porges Porges force-pushed the function/containers branch from 61b5bb7 to 43a5111 Compare July 12, 2022 21:10
@Porges Porges merged commit ec515a5 into main Jul 12, 2022
@Porges Porges deleted the function/containers branch July 12, 2022 22:01
@ghost ghost locked as resolved and limited conversation to collaborators Aug 12, 2022
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