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

Implement the node C# function #2072

Merged
merged 3 commits into from
Jun 23, 2022
Merged

Implement the node C# function #2072

merged 3 commits into from
Jun 23, 2022

Conversation

Porges
Copy link
Member

@Porges Porges commented Jun 22, 2022

Summary of the Pull Request

  1. Ports the node function from Python to C#.
  2. Adds a missing authentication check.
  3. Add validated string type PoolName for consistency with Python version.

PR Checklist

  • Applies to work item: Refactor node to C# #2068
  • 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

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

@Porges Porges force-pushed the function/node branch 3 times, most recently from f3a257a to b16ddaa Compare June 22, 2022 06:48
@Porges Porges force-pushed the function/node branch 2 times, most recently from 4b95ff1 to f7ba54a Compare June 22, 2022 21:23
Base automatically changed from function/info to main June 23, 2022 01:06
@Porges Porges merged commit 4eec0bf into main Jun 23, 2022
@Porges Porges deleted the function/node branch June 23, 2022 01:44
@ghost ghost locked as resolved and limited conversation to collaborators Jul 23, 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