Skip to content

fix typo in builder.ts:32 #3566

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

Closed
wants to merge 1 commit into from
Closed

Conversation

Damiencastro
Copy link

@Damiencastro Damiencastro commented Sep 27, 2024

I'm a novice at angular and I came across this typo. I'll be honest, I'm not 100% how to properly do linter & test stuff, but it's a pretty simple typo that's being fixed. I filled out the CLA, so please let me know if I need to do anything else!

Checklist

  • Issue number for this PR: #nnn (required)
  •    No issue number for this PR
    
  • Docs included?: (yes/no; required for all API/functional changes)
  •    No docs included, just a typo I found while debugging
    
  • Test units included?: (yes/no; required)
  •    No, since it's just a simple typo I'm hoping that I don't have to figure the tests out just yet
    
  • In a clean directory, yarn install, yarn test run successfully? (yes/no; required)
  •    No, I'll be honest, not sure what this is supposed to be. Sorry for the confusion...
    

Description

Code sample

@Damiencastro Damiencastro deleted the patch-1 branch September 27, 2024 16:05
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.

1 participant