Skip to content
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

Add style section to CONTRIBUTING.md. #188

Merged
merged 1 commit into from
Aug 26, 2016
Merged

Add style section to CONTRIBUTING.md. #188

merged 1 commit into from
Aug 26, 2016

Conversation

tswast
Copy link
Contributor

@tswast tswast commented Aug 25, 2016

I'd like to point https://github.com/GoogleCloudPlatform/Template/wiki/style.html back to the repositories where the samples should go.


Samples in this repository follow the [Google JavaScript Style
Guide](http://google.github.io/styleguide/javascriptguide.xml). This is
enforced using [JSHint](http://jshint.com/docs/).
Copy link
Member

Choose a reason for hiding this comment

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

Style is actually enforce using Semistandard

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated.

I'd like to point
https://github.com/GoogleCloudPlatform/Template/wiki/style.html back to
the repositories where the samples should go.
@codecov-io
Copy link

Current coverage is 87.49% (diff: 100%)

Merging #188 into master will not change coverage

@@             master       #188   diff @@
==========================================
  Files            58         58          
  Lines          2431       2431          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           2127       2127          
  Misses          304        304          
  Partials          0          0          

Powered by Codecov. Last update 1cb30c5...5191dfa

@jmdobry jmdobry merged commit 1a4bf80 into master Aug 26, 2016
@jmdobry jmdobry deleted the tswast-style branch August 26, 2016 04:03
ivanmkc pushed a commit that referenced this pull request Nov 3, 2022
ahrarmonsur pushed a commit that referenced this pull request Nov 8, 2022
grayside pushed a commit that referenced this pull request Nov 8, 2022
ace-n pushed a commit that referenced this pull request Nov 10, 2022
kweinmeister pushed a commit that referenced this pull request Nov 18, 2022
docs: update sample to show method inside of async function
ahrarmonsur pushed a commit that referenced this pull request Nov 18, 2022
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
jsimonweb pushed a commit to jsimonweb/nodejs-docs-samples that referenced this pull request Dec 12, 2022
irataxy pushed a commit that referenced this pull request Jan 14, 2023
🤖 I have created a release *beep* *boop*
---


## [2.2.0](googleapis/nodejs-video-transcoder@v2.1.0...v2.2.0) (2022-06-29)


### Features

* support regapic LRO ([#187](googleapis/nodejs-video-transcoder#187)) ([7924434](googleapis/nodejs-video-transcoder@7924434))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
irataxy pushed a commit that referenced this pull request Jan 18, 2023
🤖 I have created a release *beep* *boop*
---


## [2.2.0](googleapis/nodejs-video-transcoder@v2.1.0...v2.2.0) (2022-06-29)


### Features

* support regapic LRO ([#187](googleapis/nodejs-video-transcoder#187)) ([7924434](googleapis/nodejs-video-transcoder@7924434))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
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.

4 participants