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

Azure Functions - Removal of magic strings #155

Merged
merged 1 commit into from
Apr 21, 2021
Merged

Azure Functions - Removal of magic strings #155

merged 1 commit into from
Apr 21, 2021

Conversation

lechnerc77
Copy link
Contributor

The .NET functions used in the different days are build based on the standard template. However, it is best practice to avoid magic numbers/strings when defining the function name and use the nameof operator instead.

This pull request contains:

  • switch to nameof operator for FunctionName (in *.cs files as well as README.md files)
  • removal of Function1 as name
  • naming consistency over different days

@cdennig cdennig self-requested a review April 19, 2021 06:09
Copy link
Contributor

@cdennig cdennig left a comment

Choose a reason for hiding this comment

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

Makes totally sense, thank you for the contribution.

@cdennig cdennig merged commit c16bbcc into azuredevcollege:master Apr 21, 2021
@lechnerc77 lechnerc77 deleted the refactoring/azurefunctions-magic-strings branch April 21, 2021 06:35
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.

None yet

2 participants