-
Notifications
You must be signed in to change notification settings - Fork 1
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
Plugins do tests work #11
base: master
Are you sure you want to change the base?
Conversation
by removing ng-app, we placed the bootstrapping of angular when tests were ran. bootstrapping normally occurs when the page is running in a browser, not during tests. we moved the bootstrapping to a file that is not being called during tests.
The title of the Pull Request needs to be changed. |
The title of the Pull Request needs to be changed. |
The title of the Pull Request needs to be changed. |
The title of the Pull Request needs to be changed. |
We prefer small, well tested pull requests.
Please refer to Contributing to Spinnaker.
When filling out a pull request, please consider the following:
Note that we are unlikely to accept pull requests that add features without prior discussion. The best way to propose a feature is to open an issue first and discuss your ideas there before implementing them.
Helpful Tips
Add screenshots and annotations
Add a gif to show an action
GIFs are great to show an action, however they sometimes move too fast.
One way to improve them is to capture 3 pieces of content:
Put a border around a screenshot/gif
Add a border to help improve the readability.
Add expandable sections
Add expandable sections to hide away large code blocks/images or reduce the motion of a gifs