Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

docs(minerr): fill in error message descriptions #3430

Closed
wants to merge 1 commit into from

Conversation

ksheedlo
Copy link
Contributor

@ksheedlo ksheedlo commented Aug 1, 2013

Errors I've documented so far:

  • $injector:cdep
  • $injector:itkn
  • $injector:modulerr
  • $injector:nomod
  • $injector:pget
  • $injector:unpr
  • ng:areq
  • ng:cpi
  • ng:cpws
  • ngModel:noass

This is an incomplete list and should be considered a WIP until the team has gone through the error doc spreadsheet.

@mary-poppins
Copy link

Thanks for the PR!

  • Contributor signed CLA now or in the past
    • If you just signed, leave a comment here with your real name
  • PR's commit messages follow the commit message format

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@@ -2,3 +2,25 @@
@name $injector:cdep
@fullName Circular Dependency
@description

This error occurs when trying to get a service that depends on itself, either
Copy link
Contributor

Choose a reason for hiding this comment

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

This error occurs when the {@link $injector} tries to resolve a dependency that depends upon itself,...

Errors I've documented so far:
- `$injector:cdep`
- `$injector:itkn`
- `$injector:modulerr`
- `$injector:nomod`
- `$injector:pget`
- `$injector:unpr`
- `ng:areq`
- `ng:cpi`
- `ng:cpws`
- `ngModel:noass`
@ksheedlo ksheedlo closed this in e4b6a1e Aug 8, 2013
@ksheedlo
Copy link
Contributor Author

ksheedlo commented Aug 8, 2013

The PR closed itself. Interesting. Maybe we could add a pre-commit hook to make all PRs self-closing when merged. Probably that's a bad idea. Really what we need is a rebase button.

@petebacondarwin
Copy link
Contributor

If you put 'closes #1234' in your commit msg then github automatically close the relevant PR when the commit hits master.

@ksheedlo
Copy link
Contributor Author

ksheedlo commented Aug 8, 2013

I get that, but I didn't put that in my commit msg. It looks like Igor did it for me.

@IgorMinar
Copy link
Contributor

it closed because I added Closes #3430 into the commit message

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.

4 participants