Datastore namespace fix by idoshamun · Pull Request #1212 · googleapis/google-cloud-node

idoshamun · 2016-04-07T07:14:25Z

When using allocateId the namespace option was ignored due to a typo

when using allocateId the namespace option was ignored due to typo

stephenplusplus · 2016-04-07T08:10:43Z

Thank you! Could you also fix the tests? https://travis-ci.org/GoogleCloudPlatform/gcloud-node/jobs/121357907#L3059

idoshamun · 2016-04-07T09:43:35Z

👍

stephenplusplus · 2016-04-07T09:52:10Z

Thanks!

Source-Link: googleapis/synthtool@0a1b701 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:111973c0da7608bf1e60d070e5449d48826c385a6b92a56cb9203f1725d33c3d Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* refactor: Break transaction.run into smaller pieces for use with async functions and other read/write calls (#1196) * Add a unit test for commit as a non-tx The fact that using a transaction object to do a commit results in a non-transaction should be documented so that if we decide to introduce a change later where this behaves differently then it is well documented. # Conflicts: # test/transaction.ts * use linter * Write tests to capture current behavior of error When begin transaction sends back an error, we want some tests to capture what the behavior is so that when we make changes to the run function then behavior is preserved. * Add tests for passing a response A response should reach the user the right way. Add tests to make sure behavior is preserved. * run async close to working In the run function delegate calls to runAsync and use run async to make promise calls * Add runAsync to promise excludes This allows this function to return a promise instead of a promise wrapped in a promise. This makes the tests pass and behave the way they should. * Remove space * Change to use this instead of self Do not call request from self * Eliminate unused comments * Add two comments Comments should actually explain what is being done * Remove the commit test for this PR The commit test for this PR should be removed because it is not really relevant for the async run functionality. * Clarify types throughout the function The types used should be very specific so that reading the code isn’t confusing. * Add a bit more typing for clarity Add a type to the resolve function just to introduce more clarity * Change types used in the data client callback Make the types more specific in the data client callback so that it is easier to track down the signature and match against the begin transaction function. * run the linter * Add comments to clarify PR * Refactor the parsing logic out of run The parsing logic is going to be needed elsewhere so taking it apart now. * Change interface of request promise callback The interface name should be changed so that it matches what it is. It is the callback used to form a promise. * Hide data completely Change accessors to hide data completely instead of using the private modifier * PR use if/else block Eliminate the early return as suggested in the PR * Add comments to document the new functions The comments capture the parameters and return type. * Update return type in docs * Update the tests to include runAsync runAsync should be in promisfy excludes * refactor: Break transaction.run into smaller pieces for use with async functions and other read/write calls * Rename a function to be more descriptive Make sure it is explicit that we are parsing begin results. * Modify comment Modify comment so that it doesn’t reference the way the code was before. * refactor: Move commit logic and add tests that prepare for transaction function changes (#1202) * Add a unit test for commit as a non-tx The fact that using a transaction object to do a commit results in a non-transaction should be documented so that if we decide to introduce a change later where this behaves differently then it is well documented. # Conflicts: # test/transaction.ts * use linter * Write tests to capture current behavior of error When begin transaction sends back an error, we want some tests to capture what the behavior is so that when we make changes to the run function then behavior is preserved. * Add tests for passing a response A response should reach the user the right way. Add tests to make sure behavior is preserved. * run async close to working In the run function delegate calls to runAsync and use run async to make promise calls * Add runAsync to promise excludes This allows this function to return a promise instead of a promise wrapped in a promise. This makes the tests pass and behave the way they should. * Remove space * Change to use this instead of self Do not call request from self * Eliminate unused comments * Add two comments Comments should actually explain what is being done * Remove the commit test for this PR The commit test for this PR should be removed because it is not really relevant for the async run functionality. * Clarify types throughout the function The types used should be very specific so that reading the code isn’t confusing. * Add a bit more typing for clarity Add a type to the resolve function just to introduce more clarity * Change types used in the data client callback Make the types more specific in the data client callback so that it is easier to track down the signature and match against the begin transaction function. * run the linter * Add comments to clarify PR * Refactor the parsing logic out of run The parsing logic is going to be needed elsewhere so taking it apart now. * Change interface of request promise callback The interface name should be changed so that it matches what it is. It is the callback used to form a promise. * Move commit functionality to a new function The internals of commit should be moved to a new function. This way we can sandwich a commit async call between commit and runCommitAsync. * Add the commit tests Commit tests added to ensure that commit behaves the same way as before. * Fix the tests so that they pass on commit The tests should pass before we make changes to commit. * refactor one of the mocks One of the mocks does not need to be written twice * Hide data completely Change accessors to hide data completely instead of using the private modifier * PR use if/else block Eliminate the early return as suggested in the PR * Add comments to document the new functions The comments capture the parameters and return type. * Update return type in docs * Update the tests to include runAsync runAsync should be in promisfy excludes * refactor: Break transaction.run into smaller pieces for use with async functions and other read/write calls * Rename a function to be more descriptive Make sure it is explicit that we are parsing begin results. * chore(deps): update dependency @types/sinon to v17 (#1197) * chore(deps): update dependency @types/is to v0.0.25 (#1198) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@types/is](https://togithub.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/is) ([source](https://togithub.com/DefinitelyTyped/DefinitelyTyped)) | [`0.0.24` -> `0.0.25`](https://renovatebot.com/diffs/npm/@types%2fis/0.0.24/0.0.25) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@types%2fis/0.0.25?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@types%2fis/0.0.25?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@types%2fis/0.0.24/0.0.25?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@types%2fis/0.0.24/0.0.25?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - "after 9am and before 3pm" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/googleapis/nodejs-datastore).  * Modify comment Modify comment so that it doesn’t reference the way the code was before. * Create a transaction wrapper class for testing This is going to be a useful test for mocking out various layers to make sure they work the same way as before. * Clean up mocked transaction wrapper The mocked transaction wrapper should reset all mocked gapic functions and not have to be told which ones to reset. * Move test information for commit into commit block * Add gapic mocked tests for aggregation query Gapic mocked tests for run aggregation query need to be written that use the mock transaction object. * Fixing up the runQuery test * Finished the runQuery tests The runQuery tests are finished so we can use this to take apart the function from end to end. * Finished the get tests The get tests now make sure that the data coming back from the gapic layer results in the same values for users. * remove only * Add try catch blocks to handle errors in the tests try/catch logic in the test suite is added so that errors bubble up to the test runner and it is easier to see why tests failed * chore: update cloud-rad version to ^0.4.0 (#1199) Source-Link: googleapis/synthtool@1063ef3 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-nodejs:latest@sha256:e92044720ab3cb6984a70b0c6001081204375959ba3599ef6c42dd99a7783a67 Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> * Add comments and general cleanup Cleanup includes adding comments so that the tests are more readable. * Remove some redundant tests Some of these tests are captured in the various functions describe block. * Correct the comment to be more accurate The comment should talk about what the tests actually intend to do * General improvements to code quality Move declared transactionWrapper and other variables out to shorten code. Remove require and replace with import. Use more specific type. * Add data client check Add data client check and add type for data client. * Eliminate the mocked function variable The mocked function variable is not used so remove it from code. * Move begin transaction setup code The same block of code for setting up begin transaction is repeated twice. Move it into one function and use it from both before blocks. * Replace the TODO for the key Add some try blocks to make the errors more visible also. * Update description Update the description for the test to give a better explanation of what the describe block does. * Add comments to describe the purpose of signaller The comments describing the callback signaller should explain the problem it solves. * Add both dones back in The done functions were lost in the refactors. Let us add them back in. * mockedBeginTransaction should be Function Make a more specific type for mockedBeginTransaction and the functions mocked. * Use ECMA script modifier Make sure this function is completely hidden. * Remove TODOs that no longer apply The TODO statements no longer need to be followed up on so remove them. * beginTransaction type definition Remove the type definition. It is not used. * Add the setupBeginTransaction method Refactor test code to provide better error catching and also not repeat setup code for the run function. * Add a comment for private runCommit method The comment for the runCommit method just captures the fact that the function is used as a pass-through. * Modify comment to be more clear * Update comments The comments should more directly address the problem the tests are meant to solve * Eliminate redundant test code This describe block is now duplicated. Remove it. * revert comment This was a typo * refactor: Move commit logic and add tests that prepare for transaction function changes --------- Co-authored-by: Mend Renovate <renovate@whitesourcesoftware.com> Co-authored-by: gcf-owl-bot[bot] <78513119+gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> * feat: Begin transactions before each transaction read/write (#1205) * Add a unit test for commit as a non-tx The fact that using a transaction object to do a commit results in a non-transaction should be documented so that if we decide to introduce a change later where this behaves differently then it is well documented. # Conflicts: # test/transaction.ts * use linter * Write tests to capture current behavior of error When begin transaction sends back an error, we want some tests to capture what the behavior is so that when we make changes to the run function then behavior is preserved. * Add tests for passing a response A response should reach the user the right way. Add tests to make sure behavior is preserved. * run async close to working In the run function delegate calls to runAsync and use run async to make promise calls * Add runAsync to promise excludes This allows this function to return a promise instead of a promise wrapped in a promise. This makes the tests pass and behave the way they should. * Remove space * Change to use this instead of self Do not call request from self * Eliminate unused comments * Add two comments Comments should actually explain what is being done * Remove the commit test for this PR The commit test for this PR should be removed because it is not really relevant for the async run functionality. * Clarify types throughout the function The types used should be very specific so that reading the code isn’t confusing. * Add a bit more typing for clarity Add a type to the resolve function just to introduce more clarity * Change types used in the data client callback Make the types more specific in the data client callback so that it is easier to track down the signature and match against the begin transaction function. * run the linter * Add comments to clarify PR * Refactor the parsing logic out of run The parsing logic is going to be needed elsewhere so taking it apart now. * Change interface of request promise callback The interface name should be changed so that it matches what it is. It is the callback used to form a promise. * Move commit functionality to a new function The internals of commit should be moved to a new function. This way we can sandwich a commit async call between commit and runCommitAsync. * Add the mutex and transaction state etc. Still an issue with system tests. commit still needs tweaks to work with async. * Remove no-op, get commit tests working Remove the no-op, get commit tests in place. * Add mocks and additional debugging Mocks and additional debugging hooks to introspect what is going on. * Add tests for commit Make sure commit behaves the same way as before. * Add the commit tests Commit tests added to ensure that commit behaves the same way as before. * Fix the tests so that they pass on commit The tests should pass before we make changes to commit. * refactor one of the mocks One of the mocks does not need to be written twice * reverting changes to add new test on transaction * Change the promise Make the promise simpler. Change the tests to exclude functions with promisify. * Hide data completely Change accessors to hide data completely instead of using the private modifier * PR use if/else block Eliminate the early return as suggested in the PR * Add comments to document the new functions The comments capture the parameters and return type. * Update return type in docs * Update the tests to include runAsync runAsync should be in promisfy excludes * refactor: Break transaction.run into smaller pieces for use with async functions and other read/write calls * Rename a function to be more descriptive Make sure it is explicit that we are parsing begin results. * Move the mutex and the state down to derived class The mutex and state should be moved down to the derived class. We are going to override get/runQuery/runAggregateQuery there. * chore(deps): update dependency @types/sinon to v17 (#1197) * Add hook to call run before commit Add the hook to call run before calling commit in existing tests. * Add commitAsync to promisify excludes commitAsync should be resolved and then() function should be called as it was not being called before * remove the console logs * Delete run commit * Remove the private identifier Use the private modifier instead to hide data * Add withBeginTransaction withBeginTransaction will be used with all calls that begin transactions and then intend to use the mutex for locking when the begin transaction call is made. * Add another level of abstraction Add #beginWithCallback so that all the read calls can be made with one line of code. * commit async is not needed anymore * This data structure is not needed anymore * Replace types associated with run to use generics The generic parameter should be used for types with run. * Make response type more specific for parseRunAsync More specific types are better and it make code easier to read * chore(deps): update dependency @types/is to v0.0.25 (#1198) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@types/is](https://togithub.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/is) ([source](https://togithub.com/DefinitelyTyped/DefinitelyTyped)) | [`0.0.24` -> `0.0.25`](https://renovatebot.com/diffs/npm/@types%2fis/0.0.24/0.0.25) | [![age](https://developer.mend.io/api/mc/badges/age/npm/@types%2fis/0.0.25?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/@types%2fis/0.0.25?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/@types%2fis/0.0.24/0.0.25?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/@types%2fis/0.0.24/0.0.25?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: Branch creation - "after 9am and before 3pm" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/googleapis/nodejs-datastore).  * Modify comment Modify comment so that it doesn’t reference the way the code was before. * Add implementation for runQuery runQuery should make the call to begin the transaction first if that hasn’t already happened yet. * Making fixes for runAggregationQuery Fixes for run aggregation query. Still getting undefined results. * Write tests for runAggregateQuery Make sure that runAggregateQuery return results make it back to the user. * Get one test case passing for runAggregateQuery runAggregateQuery should not be excluded by promisify. Otherwise it will not return a promise, but we want it to return a promise. * remove console log clutter * Change tests for runAggregationQuery Change the test to use deep strict equal since the objects being compared will not be reference equal. * Add resolver type Eliminate some unused code * Create a transaction wrapper class for testing This is going to be a useful test for mocking out various layers to make sure they work the same way as before. * Clean up mocked transaction wrapper The mocked transaction wrapper should reset all mocked gapic functions and not have to be told which ones to reset. * Move test information for commit into commit block * Add gapic mocked tests for aggregation query Gapic mocked tests for run aggregation query need to be written that use the mock transaction object. * Fixing up the runQuery test * Finished the runQuery tests The runQuery tests are finished so we can use this to take apart the function from end to end. * Finished the get tests The get tests now make sure that the data coming back from the gapic layer results in the same values for users. * remove only * remove only and console log * Try modifications to runQuery Add modifications to runQuery to use some function to return values * Add try catch blocks to handle errors in the tests try/catch logic in the test suite is added so that errors bubble up to the test runner and it is easier to see why tests failed * Modify commit so it doesn’t run early Don’t start commit with a promise or the promise will run early. * Update get with resolver get should use the same pattern as runQuery and runAggregationQuery to use a resolver for the mutex business logic * remove #beginWithCallback #beginWithCallback is no longer used so remove it * Remove #withBeginTransaction This function is no longer used * Rename #someFunction with begin transaction #someFunction should be named differently * Fix promisify. Change to a deepStrictEqual check. A deepStrictEqual check is all that is needed in this test that currently reuses a transaction. * Add setImmediate to the tests If we add a delay to the tests then the mutex has the opportunity to unlock before running the tests. * Added some tests for call ordering We want to make sure that the calls get delivered and received in order * Pack testing tool into an object Put the testing tool in an object so that we can explore more orderings and possibilities. * Eliminate console logs, use expected order Allow expected order to be passed into the tester. This will make the object reusable. * Add a check for transaction not started yet Need a check to be sure that the transaction is not in progress if making another beginTransaction call. * Remove NOT_TRANSACTION Remove the NOT_TRANSACTION and default to NOT_STARTED * Remove only Remove only and let all unit tests run * Use an enum instead of static class members Use an enum to track transaction state. * TODOs are done * Move excludes to proper position Excludes should contain the right functions in the right order. * Simplify the run function a little bit Regroup functionality to simplify the run function * Add comments to tests Explain purpose of testing objects * Modify tests and fix a bug from the merge The merge added a bug because the pound sign wasn’t used and it should have been. * Fix error message * Comments and general cleanup Remove constructor parameter that is not used. Add comments to functions so that the tests are more readable. * Added comments for functions in transaction.ts Functions in transaction.ts need JSdoc comments. * Add a comment for aggregate queries Add JSdoc comment * Add an assertion to test The assertion should make sure the first read returns undefined. * Add tests for lookup, put, commit For calls with run and without run, do lookup, put, commit. Make sure to clean up after the tests too. * refactor the test Test does lookup, put, commit. This is done in an async function. * Add tests for put, lookup, commit A group of tests should be added for put, lookup, commit. One with run and one without run. * chore: update cloud-rad version to ^0.4.0 (#1199) Source-Link: googleapis/synthtool@1063ef3 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-nodejs:latest@sha256:e92044720ab3cb6984a70b0c6001081204375959ba3599ef6c42dd99a7783a67 Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> * Add comments and general cleanup Cleanup includes adding comments so that the tests are more readable. * Remove some redundant tests Some of these tests are captured in the various functions describe block. * Correct the comment to be more accurate The comment should talk about what the tests actually intend to do * General improvements to code quality Move declared transactionWrapper and other variables out to shorten code. Remove require and replace with import. Use more specific type. * Add data client check Add data client check and add type for data client. * Eliminate the mocked function variable The mocked function variable is not used so remove it from code. * Move begin transaction setup code The same block of code for setting up begin transaction is repeated twice. Move it into one function and use it from both before blocks. * Replace the TODO for the key Add some try blocks to make the errors more visible also. * Update description Update the description for the test to give a better explanation of what the describe block does. * Add comments to describe the purpose of signaller The comments describing the callback signaller should explain the problem it solves. * Add both dones back in The done functions were lost in the refactors. Let us add them back in. * mockedBeginTransaction should be Function Make a more specific type for mockedBeginTransaction and the functions mocked. * Use ECMA script modifier Make sure this function is completely hidden. * Remove TODOs that no longer apply The TODO statements no longer need to be followed up on so remove them. * beginTransaction type definition Remove the type definition. It is not used. * Add the setupBeginTransaction method Refactor test code to provide better error catching and also not repeat setup code for the run function. * Add a comment for private runCommit method The comment for the runCommit method just captures the fact that the function is used as a pass-through. * Modify comment to be more clear * Update comments The comments should more directly address the problem the tests are meant to solve * Eliminate redundant test code This describe block is now duplicated. Remove it. * revert comment This was a typo * refactor: Move commit logic and add tests that prepare for transaction function changes * runQuery, put, commit Another some more integration tests for runQuery, put, commit * run linter and group some tests into a describe block so that they don’t run before some new tests we are going to add. * Add some runAggregationQuery integration tests Add one of the test cases from the document (runAggregationQuery tests) * Add tests for put, runAggregationQuery, commit put, runAggregationQuery, commit tests have been added and now run properly. * Write some latency tests The latency tests measure time taken with and without using the run call. * Add logs to make latency tests run The logs will output the time required for the latency tests. * Add two tests for put, commit The two tests for put, commit should make sure that begin transaction is called. * Build requests into the transaction order tester To meet the needs we want for unit testing we must add some checker that lets us record the requests and verify that they are the right values. * Modify order testing Modify the order tester object to include more calls. Also add expected requests to the existing tests. * Create lookup, lookup, put, commit Create test for a read, read and then commit. * Fix the unit test Unit test should capture the fact that the run callback is used. * Add lookup requests When using get requests without passing consistency, we should see two lookup requests reach the gapic layer. Consistency should be removed from the test because it is not meant to reach the Gapic layer. * Remove the console logs console logs are not needed. Remove them so that the tests are cleaner. * Remove only * remove the latency tests The latency tests are not needed anymore so remove them. * Separate into a section transactions with/without A section name transactions with and without run should be used. * Remove a describe block and run linter This should make the diff a lot cleaner * Re-introduce a test that was there from before We don’t want the diff to say that we have removed tests. Therefore, add the test that was there before. * Update comment Modify the comment so that it is more objective * Add a comment for the runAggregationQuery function The comment for the aggregation query function in transaction.ts is needed here. * Simplify argument type In #withBeginTransaction there is a data type that matches the complex data type in the second argument of this function. * remove unnecessary extra variable The extra variable is not needed * Eliminate redundant definition Eliminate a definition and use the commit response data type here instead. * Add a comment for the #withBeginTransaction fn Need to clearly explain what the withBeginTransactionFunction does * Eliminate a line that was used for debugging * Add space back in * remove indent * Add empty line back in This simplifies the diff a bit * Remove ambiguous pronoun * Shorten comment * Add a comment for begin Add a comment that incorporates how this function interacts with #withBeginTransaction. * Shorten comments Don’t repeat information from the comments in the super class. Just mention the super class comments. * Type is only used once Inline the type to eliminate redundancy in the code. * Inline another promise type Inline another promise type which lets us eliminate one line of code. * Rename type to executor Executor is what it is called in the promise argument. * run linter * Simplify the data types more The types should line up for the executor in the runAsync function * Rename type to describe how it is used This name is clearer * Rename to userCallbackData. Add comments UserCallbackData is a more specific description of the type of data the generic type is. We also need comments to explain what these interfaces should represent. * resolver is a better term to use here * Change the type in the comments The type has updated since to be a more general type. * Flip if/else Flip the if else. Improvements are that the if checks for the affirmative and the longer block comes first * run linter * Add brackets Introduce brackets so that the word matching the parameter is more isolated. * Stronger type checking for runAsync Use more specific types in the runAsync function so that the compiler will complain if anything isn’t right. * Call the function fed into the promise resolver resolver should be the name used just to be consistent with all other code. * Rename variables and add comments Rename the variables so that the relationships between them are easier to understand. Add comments explaining the purpose of the variables. * Add comment for the done function * Make checkForCompletion private To avoid confusion make checkForCompletion private so that the user of the object does not need to think about using it. * Replace all events with enum values The events should not be strings. If they are strings then we might be magic string matching and this should not be done in code. * Make code more succinct Remove the need for a map. Remove occurrences of strings throughout tests and replace them with enum values. * Rename event to UserCodeEvent Call this enum UserCodeEvent as it is a better description of what the values could potentially be. * Remove the map from string names to gapic layer This map is not needed anymore because now we use the Gapic layer function event directly. * More specific type Shouldn’t just be a string. This should be a Gapic Layer function. * todo is done Assertion check is done * Eliminate need to define extra variable This just makes the code a bit more concise. * Add comments to describe test objects The comments describe Gapic layer data passed back. * There is no need to make these functions private Just inline the callbacks. A private function makes their relationship with the code that uses them confusing and each one of them is only used in one place. * Renames all callbacks to just callback Rename all callbacks to callback because that is the variable name inside the client function. * Take requests private * Take expectedRequests private expectedRequests is not used outside of the transaction order tester so we should take it private. * Take expected event order private Expected event order should be taken private. * Take event order private event order is not used outside this object * Fix some warning messages Various warnings show up in the es-lint. Fix them. * Use unknown for Gapic Layer Response Change the type to unknown to address the ES-lint error. * Get rid of any type for runQuery Use the more specific type provided by runQuery instead. * More linter fixes Addressing some of the problems in the new tests according to ES-lint. * Change parameters to match the get callback Stronger type enforcement helping us catch errors later. * Another ESLInt correction Use GetCallback here instead of any for the Get response type. * Remove query variable as it is not used * Take done private When done is passed in and saved it should only be used internally. * Eliminate unused arguments in callback Eliminate the arguments because if they are not used then they cause ES-lint errors. * Add readOnly reserved word Various properties can be readOnly as recommended by the linter. * Make type for done explicit Explicitly call out the fact that done should equal mocha’s done function. * Another place where done should be mentioned The done type should explicitly be mentioned here. * No need to access order tester to get wrapper There is no need to access the transaction wrapper to get the order tester. Just access the transaction wrapper directly. * private variable transaction wrapper Take transaction wrapper private. It does not need to be visible to the user. * Capitalize RequestType Change RequestType to be capitalized and use it whenever a request needs to be made. * request should be a more specific type RequestType should be used here. * Do info checks for error and response cases Add another assertion check. Do the check for the error and response cases. * These are actually results from runQuery * Eliminate extra line of code Add assertion check for info. Eliminate the code that stores all the results. * rename enum The word Layer is redundant. Also emphasize that the enum captures the function name. * Lookup response The variable should include the word lookup since technically that is the name of the Gapic layer endpoint. * Add comments for the transaction order tester Explain each function with a comment describing what it does. * Introduce a new push function Add a push function so that in each test it is easier to see the events line up. * Remove middle layer functions Remove functions in the transactionOrderTester and use functionality from those functions directly because then in the test it is easier to read the test’s intent. Also, add GapicRequestData type to simplify types in the test class. Shorten transactionOrderTester to just tester. * Prefer block comments The comments are multi-line so use block comments instead. * Rename event to CUSTOM_EVENT CUSTOM_EVENT should be used instead of FUNCTIONS_CALLED. FUNCTIONS_CALLED is sort of ambiguous. * Add square brackets to comments square brackets added to make the comment more readable and user friendly. * Remove unused comment * Eliminate the promiseType variable This variable is only used once. We should not define it here. * Eliminate double try/catch blocks The nested try/catch blocks are not necessary. The finally block is going to run even if it returns an error in the catch block. * Add comments to describe what T is Need to make sure the generic type is clear. * Use wrapped promise The wrapped promise and runExclusive should be used instead of releasing the mutex manually. This is safer. * Change name to callback instead Name the variable to match the parameter name being passed into runExclusive. * Eliminate extra variable Use the async function in place. Don’t define an extra variable just for one usage. * Add a test for commit first If commit is first then make sure the calls happen in the right order. * Remove a redundant check for transaction started The results won’t be any different if the first check is removed. Therefore, should remove it just because it makes the code a lot simpler. * Rename variable and use runExclusive runExclusive in the run function will get rid of excess code that is not needed. Also rename a variable to runResults to be more specific. * refactor: remove resolvers, remove promise in withBeginTransaction, simplify code calling withBeginTransaction (#1212) * Change resolvers Change the resolvers so that they all follow the same pattern. * Add a wrapWithBeginTransaction function for get Generalize the calls to withBeginTransaction. Don’t allow the error type to be undefined in UserCallbackData since the error never actually will be undefined. This is necessary to solve various compile time errors. * Use wrapWithBeginTransaction in more places commit and runQuery can now use #wrapWithBeginTransaction and this will reduce the amount of code necessary in each function respectively. * Add callbackWithError function This function replaces a common pattern that occurs in transactions where we provide a callback that accepts an error as the first argument and some generic type that extends an array of any as the rest of the arguments. * Add return type to callbackWithError Adding a return type just makes type checking stronger in the client library. We should introduce it here to make it more likely a compiler error will be thrown if the function is misused. * Delete code that is not used When playing around with code to return a standard callback, code was written that is not required anymore. Also, rename StandardCallbackArgumentsAny. * There is no need to define a separate type The UserCallbackArguments do not need a separate type if they are only used once. Define the type inline where it is needed. * TODO no longer applies * Remove TODO * Inline the error type Error or null is only used in two places so inline it here. * Add a comment for callbackWithError callbackWithError is an abstract function that needs to be explained. This comment helps with that. * Added comments to document sendUserCallbackData sendUserCallbackData needs to be described so that its purpose is clear. * Move function Move function to the right order alphabetically * Do not define a commit type only used once Do not define a separate variable for only one use. * Replace the type in the comment with Function Replace with Function to eliminate references to the generic parameter which make CI tests fail. * Just use function type for the callback No need to mention the argument types and return types. * Remove withBeginTransaction withBeginTransaction is only called by one function so remove it to condense call stack. * Simplify code usage of withBeginTransaction Eliminate the need to build a resolver for all code usages of withBeginTransaction. Eliminate the need for a promise in withBeginTransaction. Eliminate the need for a callbackWithError function. * function should not have any arguments * Remove generic Args parameter It is not used. * Remove TODOs that are not relevant anymore. * Rename to withBeginTransaction Rename the function and eliminate redundant code. * Update comments for withBeginTransaction Update the comments to reflect new parameters. * Simplify diff * Remove error as null * withBeginTransaction Move withBeginTransaction to last alphabetically * Add a comment to indicate error In the code block it is not clear what kind of error will be produced and caught in catch so a comment is added to clarify the type of error that gets produced. * Rename method to beginTxAsync Calling the method `run` is too ambiguous. Use #beginTransactionAsync instead. * Remove abstract types and inline instead With changes from before, these abstract data types are now only used once in the async function that begins a transaction. This change inlines the code that creates the promise so that these don’t have to be used at all. * Change the type of the id The id for a transaction should actually match the value returned by the server. Adding this change also allows us to be more specific with types in function arguments. * For types to align, previous txn must be flexible Previous transaction must accept the Uint8Array type because after all, that is what the id type usually is. * Set transaction type in request options The request options must accept a Uint8Array for the compiler to work now that the ids can accept this type too. * For read options, more types should be accepted The type for read options should line up with the type for other properties that store a transaction id. * Replace UserCallbackData with BeginAsyncResponse UserCallbackData is only used in one place. Therefore, the generic type is not necessary and this can be replaced with a more specific type for readability. * Rewrite description for withBeginTransaction. This description adds a few more details. * Add comments to withBeginTransaction Ensure that the use of withBeginTransaction is clear and that this function contains comments throughout it. * Make the MockedTransactionWrapper class flexible We want to test what happens when sending an error back from the begin transaction call. * Move the after hook out The after hook does the same thing for all four of the describe blocks. Move it out to reduce the amount of code required. * Add beginTransaction error test Add a test to ensure that when beginTransaction sends back an error that the error actually reaches the user when they are using a promise or a callback. * Add comments to withBeginTransaction Comments are needed in withBeginTransaction to clearly indicate the flow of control within the function. * Added a few comments for withBeginTxn Comments will help all functions that are using withBeginTransaction. --------- Co-authored-by: Mend Renovate <renovate@whitesourcesoftware.com> Co-authored-by: gcf-owl-bot[bot] <78513119+gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md --------- Co-authored-by: Mend Renovate <renovate@whitesourcesoftware.com> Co-authored-by: gcf-owl-bot[bot] <78513119+gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Source-Link: googleapis/synthtool@0a1b701 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:111973c0da7608bf1e60d070e5449d48826c385a6b92a56cb9203f1725d33c3d Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* test: modernize mocha config (#589) * fix(deps): update dependency pprof to v1.3.0 (#595) * build: add node@13 runtime (#599) Co-authored-by: Maggie Nolan <nolanmar@google.com> * chore: add GitHub actions (#600) * chore(deps): update dependency linkinator to v2 This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [linkinator](https://togithub.com/JustinBeckwith/linkinator) | devDependencies | major | [`^1.5.0` -> `^2.0.0`](https://renovatebot.com/diffs/npm/linkinator/1.8.2/2.0.1) | --- ### Release Notes <details> <summary>JustinBeckwith/linkinator</summary> ### [`v2.0.1`](https://togithub.com/JustinBeckwith/linkinator/releases/v2.0.1) [Compare Source](https://togithub.com/JustinBeckwith/linkinator/compare/v2.0.0...v2.0.1) ##### Bug Fixes - ignore prefetch and connect for link ([#145](https://togithub.com/JustinBeckwith/linkinator/issues/145)) ([154eb2b](https://togithub.com/JustinBeckwith/linkinator/commit/154eb2b6c3ac8dab8f55e7d05566ad7aff1e5d9a)) ### [`v2.0.0`](https://togithub.com/JustinBeckwith/linkinator/releases/v2.0.0) [Compare Source](https://togithub.com/JustinBeckwith/linkinator/compare/v1.8.2...v2.0.0) - build!: drop support for node.js 8.x ([#142](https://togithub.com/JustinBeckwith/linkinator/issues/142)) ([85642bd](https://togithub.com/JustinBeckwith/linkinator/commit/85642bdb1ccead89dc6d25f7891054f28e6ce609)), closes [#142](https://togithub.com/JustinBeckwith/linkinator/issues/142) ##### BREAKING CHANGES - This module now requires Node.js 10.x or greater. Please upgrade to a LTS release of node.js. </details> --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is stale, or if you tick the rebase/retry checkbox below. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * doc: remove Circle CI status badge (#598) * fix(deps): update dependency @types/semver to v7 (#587) * chore(deps): update dependency sinon to v9 (#602) * chore(deps): update dependency nock to v12 * build: add publish.yml enabling GitHub app for publishes * chore: update jsdoc.js (#604) * build: update linkinator config (#606) * build(tests): fix coveralls and enable build cop (#607) * chore: update github actions configuration (#611) This PR was generated using Autosynth. :rainbow: <details><summary>Log from Synthtool</summary> ``` 2020-03-22 04:09:07,897 synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py. .eslintignore .eslintrc.yml .github/ISSUE_TEMPLATE/bug_report.md .github/ISSUE_TEMPLATE/feature_request.md .github/ISSUE_TEMPLATE/support_request.md .github/PULL_REQUEST_TEMPLATE.md .github/publish.yml .github/release-please.yml .github/workflows/ci.yaml .kokoro/common.cfg .kokoro/continuous/node10/common.cfg .kokoro/continuous/node10/docs.cfg .kokoro/continuous/node10/lint.cfg .kokoro/continuous/node10/samples-test.cfg .kokoro/continuous/node10/system-test.cfg .kokoro/continuous/node10/test.cfg .kokoro/continuous/node12/common.cfg .kokoro/continuous/node12/test.cfg .kokoro/continuous/node8/common.cfg .kokoro/continuous/node8/test.cfg .kokoro/docs.sh .kokoro/lint.sh .kokoro/presubmit/node10/common.cfg .kokoro/presubmit/node10/docs.cfg .kokoro/presubmit/node10/lint.cfg .kokoro/presubmit/node10/samples-test.cfg .kokoro/presubmit/node10/system-test.cfg .kokoro/presubmit/node10/test.cfg .kokoro/presubmit/node12/common.cfg .kokoro/presubmit/node12/test.cfg .kokoro/presubmit/node8/common.cfg .kokoro/presubmit/node8/test.cfg .kokoro/presubmit/windows/common.cfg .kokoro/presubmit/windows/test.cfg .kokoro/publish.sh .kokoro/release/docs.cfg .kokoro/release/docs.sh .kokoro/release/publish.cfg .kokoro/samples-test.sh .kokoro/system-test.sh .kokoro/test.bat .kokoro/test.sh .kokoro/trampoline.sh .mocharc.js .nycrc .prettierignore .prettierrc CODE_OF_CONDUCT.md CONTRIBUTING.md LICENSE README.md codecov.yaml renovate.json samples/README.md fatal: Invalid revision range dd7cd93888cbeb1d4c56a1ca814491c7813160e8..HEAD 2020-03-22 04:09:08,392 synthtool > Wrote metadata to synth.metadata. ``` </details> * build: set AUTOSYNTH_MULTIPLE_COMMITS=true for context aware commits (#614) * chore(deps): update dependency @types/sinon to v9 (#615) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [@types/sinon](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | devDependencies | major | [`^7.0.12` -> `^9.0.0`](https://renovatebot.com/diffs/npm/@types%2fsinon/7.5.2/9.0.0) | --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * build: update templates (#616) * chore: remove duplicate mocha config (#617) * fix: apache license URL (#468) (#618) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/9c94202f-63a5-4df0-9d76-871a00f99b85/targets * fix(deps): update dependency gcp-metadata to v4 (#609) * fix(deps): update dependency pretty-ms to v6 (#596) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [pretty-ms](https://togithub.com/sindresorhus/pretty-ms) | dependencies | major | [`^5.0.0` -> `^6.0.0`](https://renovatebot.com/diffs/npm/pretty-ms/5.1.0/6.0.1) | --- ### Release Notes <details> <summary>sindresorhus/pretty-ms</summary> ### [`v6.0.1`](https://togithub.com/sindresorhus/pretty-ms/releases/v6.0.1) [Compare Source](https://togithub.com/sindresorhus/pretty-ms/compare/v6.0.0...v6.0.1) - Fix spacing when using `colonNotation` option ([#47](https://togithub.com/sindresorhus/pretty-ms/issues/47)) [`0de4dc3`](https://togithub.com/sindresorhus/pretty-ms/commit/0de4dc3) ### [`v6.0.0`](https://togithub.com/sindresorhus/pretty-ms/releases/v6.0.0) [Compare Source](https://togithub.com/sindresorhus/pretty-ms/compare/v5.1.0...v6.0.0) ##### Breaking - Require Node.js 10 [`46433b4`](https://togithub.com/sindresorhus/pretty-ms/commit/46433b4) - Remove tilde prefix `~` from `compact` and `unitCount` option output [`4e3c6a6`](https://togithub.com/sindresorhus/pretty-ms/commit/4e3c6a6) [`737628f`](https://togithub.com/sindresorhus/pretty-ms/commit/737628f) ##### Fixes - Fix milliseconds rounding inconsistency ([#41](https://togithub.com/sindresorhus/pretty-ms/issues/41)) [`f48b81c`](https://togithub.com/sindresorhus/pretty-ms/commit/f48b81c) </details> --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * fix(deps): update dependency @google-cloud/common to v3 (#613) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [@google-cloud/common](https://togithub.com/googleapis/nodejs-common) | dependencies | major | [`^2.1.2` -> `^3.0.0`](https://renovatebot.com/diffs/npm/@google-cloud%2fcommon/2.4.0/3.0.0) | --- ### Release Notes <details> <summary>googleapis/nodejs-common</summary> ### [`v3.0.0`](https://togithub.com/googleapis/nodejs-common/blob/master/CHANGELOG.md#300-httpswwwgithubcomgoogleapisnodejs-commoncomparev240v300-2020-03-26) [Compare Source](https://togithub.com/googleapis/nodejs-common/compare/v2.4.0...v3.0.0) ##### ⚠ BREAKING CHANGES - drop support for node.js 8 ([#554](https://togithub.com/googleapis/nodejs-common/issues/554)) - remove support for custom promises ([#541](https://togithub.com/googleapis/nodejs-common/issues/541)) ##### Features - add progress events ([#540](https://www.github.com/googleapis/nodejs-common/issues/540)) ([1834059](https://www.github.com/googleapis/nodejs-common/commit/18340596ecb61018e5427371b9b5a120753ec003)) ##### Bug Fixes - remove support for custom promises ([#541](https://www.github.com/googleapis/nodejs-common/issues/541)) ([ecf1c16](https://www.github.com/googleapis/nodejs-common/commit/ecf1c167927b609f13dc4fbec1954ff3a2765344)) - **deps:** update dependency [@google-cloud/projectify](https://togithub.com/google-cloud/projectify) to v2 ([#553](https://www.github.com/googleapis/nodejs-common/issues/553)) ([23030a2](https://www.github.com/googleapis/nodejs-common/commit/23030a25783cd091f4720c25a15416c91e7bd0a0)) - **deps:** update dependency [@google-cloud/promisify](https://togithub.com/google-cloud/promisify) to v2 ([#552](https://www.github.com/googleapis/nodejs-common/issues/552)) ([63175e0](https://www.github.com/googleapis/nodejs-common/commit/63175e0c4504020466a95e92c2449bdb8ac47546)) ##### Miscellaneous Chores - drop support for node.js 8 ([#554](https://www.github.com/googleapis/nodejs-common/issues/554)) ([9f41047](https://www.github.com/googleapis/nodejs-common/commit/9f410477432893f68e57b5eeb31a068a3d8ef52f)) </details> --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * build: remove unused codecov config (#622) * refactor: use consistent directory structure (#621) * build!: require node.js 10.x and up (#623) * fix(deps): update dependency pprof to v2 (#619) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [pprof](https://togithub.com/google/pprof-nodejs) | dependencies | major | [`1.3.0` -> `2.0.0`](https://renovatebot.com/diffs/npm/pprof/1.3.0/2.0.0) | --- ### Release Notes <details> <summary>google/pprof-nodejs</summary> ### [`v2.0.0`](https://togithub.com/google/pprof-nodejs/releases/v2.0.0) [Compare Source](https://togithub.com/google/pprof-nodejs/compare/v1.3.0...v2.0.0) This change drops support for Node 8. #### Breaking changes [`f44ee9a`](https://togithub.com/google/pprof-nodejs/commit/f44ee9aaf834baf457417a40e05adc76c4f199ad) chore!: Drop support for Node 8 ([#111](https://togithub.com/google/pprof-nodejs/issues/111)) #### Dependencies [`17171e1`](https://togithub.com/google/pprof-nodejs/commit/17171e1e55ecd68d43c88491cc7efbf79be0291f) chore(deps): update dependency linkinator to v2 ([#102](https://togithub.com/google/pprof-nodejs/issues/102)) [`086c71c`](https://togithub.com/google/pprof-nodejs/commit/086c71c34ad3742808fbc2aa23329214cf2b5970) fix(deps): update dependency pify to v5 ([#105](https://togithub.com/google/pprof-nodejs/issues/105)) [`d54480b`](https://togithub.com/google/pprof-nodejs/commit/d54480b5b2a0095523bdeb9c08b976b4f28891bb) chore(deps): update dependency sinon to v9 ([#107](https://togithub.com/google/pprof-nodejs/issues/107)) [`d25c53b`](https://togithub.com/google/pprof-nodejs/commit/d25c53b9a0c8cce30e3bd7eb028b5380daf2be1b) chore(deps): update dependency typescript to ~3.8.0 ([#108](https://togithub.com/google/pprof-nodejs/issues/108)) [`b8ec099`](https://togithub.com/google/pprof-nodejs/commit/b8ec0991c586451f222fff54b76746bd7d6a826c) chore(deps): update golang docker tag to v1.14 ([#109](https://togithub.com/google/pprof-nodejs/issues/109)) [`a61df53`](https://togithub.com/google/pprof-nodejs/commit/a61df539e5fb5e74edbe374c77d2dbedbe6c32a0) chore(deps): update dependency [@types/mocha](https://togithub.com/types/mocha) to v7 ([#100](https://togithub.com/google/pprof-nodejs/issues/100)) #### Internal [`814e89d`](https://togithub.com/google/pprof-nodejs/commit/814e89dc3dec7f155c5752967d80f15d88933ad9) chore: remove Kokoro unit tests ([#110](https://togithub.com/google/pprof-nodejs/issues/110)) [`c7e213b`](https://togithub.com/google/pprof-nodejs/commit/c7e213b3d5e349f8edb65367035a88ce374427e0) chore: start running unit tests with GitHub workflows ([#106](https://togithub.com/google/pprof-nodejs/issues/106)) </details> --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * chore: clean-up system tests and README.md after dropping support for node 8 (#625) * feat: package is now GA (#627) * build: allow profiler to have custom actions (#633) * chore: retry install go modules in integration test (#632) * chore: run system tests in GitHub workflow * chore: retry install go modules in integration test * do not retry go mod init * chore: release 4.0.0 (#586) * updated CHANGELOG.md [ci skip] * updated package.json [ci skip] * updated samples/package.json [ci skip] Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Maggie Nolan <nolanmar@google.com> * fix(deps): update dependency protobufjs to ~6.9.0 (#634) * chore: install newer version of nvm in integration test (#639) * chore(deps): update dependency tmp to v0.2.0 (#641) * fix(deps): update dependency pretty-ms to v7 (#642) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [pretty-ms](https://togithub.com/sindresorhus/pretty-ms) | dependencies | major | [`^6.0.0` -> `^7.0.0`](https://renovatebot.com/diffs/npm/pretty-ms/6.0.1/7.0.0) | --- ### Release Notes <details> <summary>sindresorhus/pretty-ms</summary> ### [`v7.0.0`](https://togithub.com/sindresorhus/pretty-ms/releases/v7.0.0) [Compare Source](https://togithub.com/sindresorhus/pretty-ms/compare/v6.0.1...v7.0.0) ##### Breaking - Always floor time instead of rounding up ([#49](https://togithub.com/sindresorhus/pretty-ms/issues/49)) [`20cbdaf`](https://togithub.com/sindresorhus/pretty-ms/commit/20cbdaf) It's probably not breaking to most users, but if you depend on the exact output, in for example, unit tests, the output might have changed slightly. </details> --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * chore: compile then run golang test in e2e test (#645) * chore(deps): update dependency tmp to v0.2.1 (#644) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [tmp](https://togithub.com/raszi/node-tmp) | devDependencies | patch | [`0.2.0` -> `0.2.1`](https://renovatebot.com/diffs/npm/tmp/0.2.0/0.2.1) | | [@types/tmp](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | devDependencies | minor | [`0.1.0` -> `0.2.0`](https://renovatebot.com/diffs/npm/@types%2ftmp/0.1.0/0.2.0) | --- ### Release Notes <details> <summary>raszi/node-tmp</summary> ### [`v0.2.1`](https://togithub.com/raszi/node-tmp/blob/master/CHANGELOG.md#v021-2020-04-28) [Compare Source](https://togithub.com/raszi/node-tmp/compare/v0.2.0...7ae22ed2d56c10d425a66e99fe8bc10c925442e6) ##### :rocket: Enhancement - [#252](https://togithub.com/raszi/node-tmp/pull/252) Closes [#250](https://togithub.com/raszi/node-tmp/issues/250): introduce tmpdir option for overriding the system tmp dir ([@silkentrance](https://togithub.com/silkentrance)) ##### :house: Internal - [#253](https://togithub.com/raszi/node-tmp/pull/253) Closes [#191](https://togithub.com/raszi/node-tmp/issues/191): generate changelog from pull requests using lerna-changelog ([@silkentrance](https://togithub.com/silkentrance)) ##### Committers: 1 - Carsten Klein ([@silkentrance](https://togithub.com/silkentrance)) </details> --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * chore: add backoff to retry function used in backoff test (#650) * fix: malformed tsconfig causing broken tests #640 (#647) * chore(deps): update dependency js-green-licenses to v2 (#646) * chore: update npm scripts and synth.py (#640) Update npm scripts: add clean, prelint, prefix; make sure that lint and fix are set properly. Use post-process feature of synthtool. Co-authored-by: Justin Beckwith <justin.beckwith@gmail.com> * chore: fix integration test dependency installation retries (#636) * fix(deps): update dependency teeny-request to v7 (#652) * build: migrate to secret manager (#653) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/9b55eba7-85ee-48d5-a737-8b677439db4d/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/1c92077459db3dc50741e878f98b08c6261181e0 * chore(deps): update dependency mocha to v8 (#654) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [mocha](https://mochajs.org/) ([source](https://togithub.com/mochajs/mocha)) | devDependencies | major | [`^7.0.0` -> `^8.0.0`](https://renovatebot.com/diffs/npm/mocha/7.2.0/8.0.1) | --- ### Release Notes <details> <summary>mochajs/mocha</summary> ### [`v8.0.1`](https://togithub.com/mochajs/mocha/blob/master/CHANGELOG.md#801--2020-06-10) [Compare Source](https://togithub.com/mochajs/mocha/compare/v8.0.0...v8.0.1) The obligatory patch after a major. #### :bug: Fixes - [#4328](https://togithub.com/mochajs/mocha/issues/4328): Fix `--parallel` when combined with `--watch` ([**@boneskull**](https://togithub.com/boneskull)) ### [`v8.0.0`](https://togithub.com/mochajs/mocha/blob/master/CHANGELOG.md#800--2020-06-10) [Compare Source](https://togithub.com/mochajs/mocha/compare/v7.2.0...v8.0.0) In this major release, Mocha adds the ability to _run tests in parallel_. Better late than never! Please note the **breaking changes** detailed below. Let's welcome [**@giltayar**](https://togithub.com/giltayar) and [**@nicojs**](https://togithub.com/nicojs) to the maintenance team! #### :boom: Breaking Changes - [#4164](https://togithub.com/mochajs/mocha/issues/4164): **Mocha v8.0.0 now requires Node.js v10.0.0 or newer.** Mocha no longer supports the Node.js v8.x line ("Carbon"), which entered End-of-Life at the end of 2019 ([**@UlisesGascon**](https://togithub.com/UlisesGascon)) - [#4175](https://togithub.com/mochajs/mocha/issues/4175): Having been deprecated with a warning since v7.0.0, **`mocha.opts` is no longer supported** ([**@juergba**](https://togithub.com/juergba)) :sparkles: **WORKAROUND:** Replace `mocha.opts` with a [configuration file](https://mochajs.org/#configuring-mocha-nodejs). - [#4260](https://togithub.com/mochajs/mocha/issues/4260): Remove `enableTimeout()` (`this.enableTimeout()`) from the context object ([**@craigtaub**](https://togithub.com/craigtaub)) :sparkles: **WORKAROUND:** Replace usage of `this.enableTimeout(false)` in your tests with `this.timeout(0)`. - [#4315](https://togithub.com/mochajs/mocha/issues/4315): The `spec` option no longer supports a comma-delimited list of files ([**@juergba**](https://togithub.com/juergba)) :sparkles: **WORKAROUND**: Use an array instead (e.g., `"spec": "foo.js,bar.js"` becomes `"spec": ["foo.js", "bar.js"]`). - [#4309](https://togithub.com/mochajs/mocha/issues/4309): Drop support for Node.js v13.x line, which is now End-of-Life ([**@juergba**](https://togithub.com/juergba)) - [#4282](https://togithub.com/mochajs/mocha/issues/4282): `--forbid-only` will throw an error even if exclusive tests are avoided via `--grep` or other means ([**@arvidOtt**](https://togithub.com/arvidOtt)) - [#4223](https://togithub.com/mochajs/mocha/issues/4223): The context object's `skip()` (`this.skip()`) in a "before all" (`before()`) hook will no longer execute subsequent sibling hooks, in addition to hooks in child suites ([**@juergba**](https://togithub.com/juergba)) - [#4178](https://togithub.com/mochajs/mocha/issues/4178): Remove previously soft-deprecated APIs ([**@wnghdcjfe**](https://togithub.com/wnghdcjfe)): - `Mocha.prototype.ignoreLeaks()` - `Mocha.prototype.useColors()` - `Mocha.prototype.useInlineDiffs()` - `Mocha.prototype.hideDiff()` #### :tada: Enhancements - [#4245](https://togithub.com/mochajs/mocha/issues/4245): Add ability to run tests in parallel for Node.js (see [docs](https://mochajs.org/#parallel-tests)) ([**@boneskull**](https://togithub.com/boneskull)) :exclamation: See also [#4244](https://togithub.com/mochajs/mocha/issues/4244); [Root Hook Plugins (docs)](https://mochajs.org/#root-hook-plugins) -- _root hooks must be defined via Root Hook Plugins to work in parallel mode_ - [#4304](https://togithub.com/mochajs/mocha/issues/4304): `--require` now works with ES modules ([**@JacobLey**](https://togithub.com/JacobLey)) - [#4299](https://togithub.com/mochajs/mocha/issues/4299): In some circumstances, Mocha can run ES modules under Node.js v10 -- _use at your own risk!_ ([**@giltayar**](https://togithub.com/giltayar)) #### :book: Documentation - [#4246](https://togithub.com/mochajs/mocha/issues/4246): Add documentation for parallel mode and Root Hook plugins ([**@boneskull**](https://togithub.com/boneskull)) #### :bug: Fixes (All bug fixes in Mocha v8.0.0 are also breaking changes, and are listed above) </details> --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * chore(nodejs_templates): add script logging to node_library populate-secrets.sh (#655) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/e306327b-605f-4c07-9420-c106e40c47d5/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/e7034945fbdc0e79d3c57f6e299e5c90b0f11469 * chore: update node issue template (#656) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/37f383f8-7560-459e-b66c-def10ff830cb/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/b10590a4a1568548dd13cfcea9aa11d40898144b * build: add config .gitattributes (#657) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/2a81bca4-7abd-4108-ac1f-21340f858709/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/dc9caca650c77b7039e2bbc3339ffb34ae78e5b7 * chore(deps): update dependency nock to v13 (#658) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [nock](https://togithub.com/nock/nock) | devDependencies | major | [`^12.0.0` -> `^13.0.0`](https://renovatebot.com/diffs/npm/nock/12.0.3/13.0.0) | --- ### Release Notes <details> <summary>nock/nock</summary> ### [`v13.0.0`](https://togithub.com/nock/nock/releases/v13.0.0) [Compare Source](https://togithub.com/nock/nock/compare/v12.0.3...v13.0.0) See the [Migration Guide](https://togithub.com/nock/nock/blob/75507727cf09a0b7bf0aa7ebdf3621952921b82e/migration_guides/migrating_to_13.md) ##### Breaking changes 1. `Scope.log` has been removed. Use the `debug` library when [debugging](https://togithub.com/nock/nock#debugging) failed matches. 2. `socketDelay` has been removed. Use [`delayConnection`](https://togithub.com/nock/nock#delay-the-connection) instead. 3. `delay`, `delayConnection`, and `delayBody` are now setters instead of additive. 4. [When recording](https://togithub.com/nock/nock#recording), skipping body matching using `*` is no longer supported by `nock.define`. Set the definition body to `undefined` instead. 5. `ClientRequest.abort()` has been updated to align with Node's native behavior. This could be considered a feature, however, it created some subtle differences that are not backwards compatible. Refer to the migration guide for details. 6. Playback of a mocked responses will now never happen until the 'socket' event is emitted. </details> --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * chore: update CODEOWNERS (#659) * fix: typeo in nodejs .gitattribute (#661) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/cc99acfa-05b8-434b-9500-2f6faf2eaa02/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/799d8e6522c1ef7cb55a70d9ea0b15e045c3d00b * chore: release 4.0.1 (#635) :robot: I have created a release \*beep\* \*boop\* --- ### [4.0.1](https://www.github.com/googleapis/cloud-profiler-nodejs/compare/v4.0.0...v4.0.1) (2020-07-09) ### Bug Fixes * **deps:** update dependency pretty-ms to v7 ([#642](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/642)) ([f69c7a7](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/f69c7a73b17c150c2b523412e430b5d1ac03e12a)) * **deps:** update dependency protobufjs to ~6.9.0 ([#634](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/634)) ([a90149c](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/a90149c4f91630d75bb41fab145713637736d21c)) * malformed tsconfig causing broken tests [#640](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/640) ([#647](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/647)) ([09c19c8](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/09c19c88a0ca137b7970c386730b145b66b77ec3)) * **deps:** update dependency teeny-request to v7 ([#652](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/652)) ([b46eb4f](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/b46eb4f2552871f405579100e4e916dbde5c60d8)) * typeo in nodejs .gitattribute ([#661](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/661)) ([92f46ac](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/92f46ac22a510c7bab05549b83779dd4f60096b7)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). * chore(deps): update dependency @types/mocha to v8 (#664) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [@types/mocha](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | devDependencies | major | [`^7.0.0` -> `^8.0.0`](https://renovatebot.com/diffs/npm/@types%2fmocha/7.0.2/8.0.0) | --- ### Renovate configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#googleapis/cloud-profiler-nodejs). * chore: add backoff integration test (#651) * chore: add backoff integration test * stop running backoff test as presubmit and clarify comment * fix(deps): update dependency protobufjs to ~6.10.0 (#665) * chore: delete Node 8 presubmit tests (#666) * chore: release 4.0.2 (#667) * fix(deps): update dependency parse-duration to 0.4.4 (#668) * build: fix typo in publish (#674) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/5b03461e-47c0-40e8-a8ad-c465ee146cc5/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/388e10f5ae302d3e8de1fac99f3a95d1ab8f824a Source-Link: https://github.com/googleapis/synthtool/commit/d82deccf657a66e31bd5da9efdb96c6fa322fc7e * chore: add config files for cloud-rad for node.js (#675) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/5e903fff-57bb-4395-bb94-8b4d1909dbf6/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/21f1470ecd01424dc91c70f1a7c798e4e87d1eec * chore: add dev dependencies for cloud-rad ref docs (#676) * chore: backoff test should run as a continuous, not presubmit, test (#678) * build: rename _toc to toc (#677) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/940354f9-15cd-4361-bbf4-dc9af1426979/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/99c93fe09f8c1dca09dfc0301c8668e3a70dd796 * fix: move gitattributes files to node templates (#679) Source-Author: F. Hinkelmann <franziska.hinkelmann@gmail.com> Source-Date: Thu Jul 23 01:45:04 2020 -0400 Source-Repo: googleapis/synthtool Source-Sha: 3a00b7fea8c4c83eaff8eb207f530a2e3e8e1de3 Source-Link: https://github.com/googleapis/synthtool/commit/3a00b7fea8c4c83eaff8eb207f530a2e3e8e1de3 * chore(node): fix kokoro build path for cloud-rad (#680) Source-Author: F. Hinkelmann <franziska.hinkelmann@gmail.com> Source-Date: Wed Jul 29 00:28:42 2020 -0400 Source-Repo: googleapis/synthtool Source-Sha: 89d431fb2975fc4e0ed24995a6e6dfc8ff4c24fa Source-Link: https://github.com/googleapis/synthtool/commit/89d431fb2975fc4e0ed24995a6e6dfc8ff4c24fa * docs: add links to the CHANGELOG from the README.md for Java and Node (#681) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/7b446397-88f3-4463-9e7d-d2ce7069989d/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/5936421202fb53ed4641bcb824017dd393a3dbcc * chore: remove references to Stackdriver (#682) * build: --credential-file-override is no longer required (#685) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/4de22315-84b1-493d-8da2-dfa7688128f5/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/94421c47802f56a44c320257b2b4c190dc7d6b68 * chore: update cloud rad kokoro build job (#686) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/b742586e-df31-4aac-8092-78288e9ea8e7/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/bd0deaa1113b588d70449535ab9cbf0f2bd0e72f * build: perform publish using Node 12 (#687) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/c36c6dbc-ab79-4f17-b70b-523b420b2a70/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/5747555f7620113d9a2078a48f4c047a99d31b3e * chore: start tracking obsolete files (#688) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/7a1b0b96-8ddb-4836-a1a2-d2f73b7e6ffe/targets - [ ] To automatically regenerate this PR, check this box. * build: move system and samples test from Node 10 to Node 12 (#689) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/ba2d388f-b3b2-4ad7-a163-0c6b4d86894f/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/05de3e1e14a0b07eab8b474e669164dbd31f81fb * build: track flaky tests for "nightly", add new secrets for tagging (#690) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/742277d3-5124-43f1-b35f-37558686e819/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/8cf6d2834ad14318e64429c3b94f6443ae83daf9 * chore: simplify e2e test npm output (#692) * chore: add sync repo settings (#693) * build(test): recursively find test files; fail on unsupported dependency versions (#694) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/b07e7603-6d2e-453b-934d-7b03566ffcfd/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/fdd03c161003ab97657cc0218f25c82c89ddf4b6 * chore: release 4.0.3 (#672) Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> * chore: update bucket for cloud-rad (#695) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/50d3d05e-d0e3-4b19-aa85-502f92e4470a/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/079dcce498117f9570cebe6e6cff254b38ba3860 * build(node_library): migrate to Trampoline V2 (#696) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/e58201ce-61aa-4909-a1c6-a4b8f6e73db0/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/0c868d49b8e05bc1f299bc773df9eb4ef9ed96e9 * fix: Update engines to prevent agent from being used with versions of Node.js where v8 profilers have memory leaks (#699) * Revert "fix: Update engines to prevent agent from getting used when it will cause memory leak." (#706) * Revert "fix: Update engines to prevent agent from being used with versions of Node.js where v8 profilers have memory leaks (#699)" This reverts commit 160d1f68c657f6ce18b8cea921470d1b4482619f. * address comment * chore(deps): update pprof to v3.0.0 (#708) * chore: clean up Node.js TOC for cloud-rad (#705) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/d032c795-21ea-4ddd-9af9-5229b7741969/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/901ddd44e9ef7887ee681b9183bbdea99437fdcc Source-Link: https://github.com/googleapis/synthtool/commit/f96d3b455fe27c3dc7bc37c3c9cd27b1c6d269c8 * feat: add support for Node 14 (#709) * chore: release 4.1.0 (#711) Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> * docs: updated code of conduct (#712) * chore(docs): update code of conduct of synthtool and templates Source-Author: Christopher Wilcox <crwilcox@google.com> Source-Date: Thu Oct 22 14:22:01 2020 -0700 Source-Repo: googleapis/synthtool Source-Sha: 5f6ef0ec5501d33c4667885b37a7685a30d41a76 Source-Link: https://github.com/googleapis/synthtool/commit/5f6ef0ec5501d33c4667885b37a7685a30d41a76 * build(node): add KOKORO_BUILD_ARTIFACTS_SUBDIR to env Source-Author: Benjamin E. Coe <bencoe@google.com> Source-Date: Mon Nov 2 15:56:09 2020 -0500 Source-Repo: googleapis/synthtool Source-Sha: ba9918cd22874245b55734f57470c719b577e591 Source-Link: https://github.com/googleapis/synthtool/commit/ba9918cd22874245b55734f57470c719b577e591 * build: use standard CI config (#710) Co-authored-by: Benjamin E. Coe <bencoe@google.com> * docs: spelling correction for "targetting" (#713) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/51ff06be-2295-4bd1-a377-6ea423db5d24/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/15013eff642a7e7e855aed5a29e6e83c39beba2a * docs: add instructions for authenticating for system tests (#714) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/f858a143-daac-4e50-b9ae-219abe9981ce/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/363fe305e9ce34a6cd53951c6ee5f997094b54ee * chore(deps): update dependency js-green-licenses to v3 (#716) * chore: make git clone idempotent so that it retries properly (#718) * chore: switch to the proftest API that logs the VM output per test (#720) * refactor(nodejs): move build cop to flakybot (#719) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/1ff854df-6525-4c07-b2b3-1ffce863e3f6/targets - [ ] To automatically regenerate this PR, check this box. Source-Link: https://github.com/googleapis/synthtool/commit/57c23fa5705499a4181095ced81f0ee0933b64f6 * chore: update CODEOWNERS config (#721) * fix(deps): update dependency delay to v5 (#722) * fix(deps): update dependency parse-duration to v1 (#725) * chore: release 4.1.1 (#723) Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> * chore(deps): update dependency sinon to v10 (#726) [![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [sinon](https://sinonjs.org/) ([source](https://togithub.com/sinonjs/sinon)) | [`^9.0.0` -> `^10.0.0`](https://renovatebot.com/diffs/npm/sinon/9.2.4/10.0.0) | [![age](https://badges.renovateapi.com/packages/npm/sinon/10.0.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/sinon/10.0.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/sinon/10.0.0/compatibility-slim/9.2.4)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/sinon/10.0.0/confidence-slim/9.2.4)](https://docs.renovatebot.com/merge-confidence/) | | [@types/sinon](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | [`^9.0.0` -> `^10.0.0`](https://renovatebot.com/diffs/npm/@types%2fsinon/9.0.11/10.0.0) | [![age](https://badges.renovateapi.com/packages/npm/@types%2fsinon/10.0.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/@types%2fsinon/10.0.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/@types%2fsinon/10.0.0/compatibility-slim/9.0.11)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/@types%2fsinon/10.0.0/confidence-slim/9.0.11)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>sinonjs/sinon</summary> ### [`v10.0.0`](https://togithub.com/sinonjs/sinon/blob/master/CHANGELOG.md#1000--2021-03-22) [Compare Source](https://togithub.com/sinonjs/sinon/compare/v9.2.4...v10.0.0) ================== - Upgrade nise to 4.1.0 - Use [@sinonjs/eslint-config](https://togithub.com/sinonjs/eslint-config)[@4](https://togithub.com/4) => Adopts ES2017 => Drops support for IE 11, Legacy Edge and legacy Safari </details> --- ### Configuration :date: **Schedule**: "after 9am and before 3pm" (UTC). :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied. :recycle: **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. :no_bell: **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, check this box. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/cloud-profiler-nodejs). * chore: regenerate common templates (#728) This PR was generated using Autosynth. :rainbow: Synth log will be available here: https://source.cloud.google.com/results/invocations/1458310e-41a7-4ec2-87e4-35b926aec063/targets - [ ] To automatically regenerate this PR, check this box. (May take up to 24 hours.) Source-Link: https://github.com/googleapis/synthtool/commit/c6706ee5d693e9ae5967614170732646590d8374 Source-Link: https://github.com/googleapis/synthtool/commit/b33b0e2056a85fc2264b294f2cf47dcd45e95186 Source-Link: https://github.com/googleapis/synthtool/commit/898b38a6f4fab89a76dfb152480bb034a781331b * build: warn uses when they edit auto-generated files (#732) * fix(deps): update dependency pprof to v3.1.0 (#731) * fix(deps): update dependency protobufjs to ~6.11.0 (#733) * chore: release 4.1.2 (#734) :robot: I have created a release \*beep\* \*boop\* --- ### [4.1.2](https://www.github.com/googleapis/cloud-profiler-nodejs/compare/v4.1.1...v4.1.2) (2021-05-05) ### Bug Fixes * **deps:** update dependency pprof to v3.1.0 ([#731](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/731)) ([ba96e49](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/ba96e49a2d254d713d55f16bedbf6a5268500801)) * **deps:** update dependency protobufjs to ~6.11.0 ([#733](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/733)) ([33abbeb](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/33abbebe1e6424658a66cdb728b93875d4edadd3)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). * chore: migrate to owl bot (#737) * chore: migrate to owl bot * chore: copy files from googleapis-gen 397c0bfd367a2427104f988d5329bc117caafd95 * chore: run the post processor * lint * fix: lint Co-authored-by: Justin Beckwith <justin.beckwith@gmail.com> * chore(deps): update dependency @types/node to v14 (#738) [![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@types/node](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | [`^10.0.3` -> `^14.0.0`](https://renovatebot.com/diffs/npm/@types%2fnode/10.17.60/14.17.0) | [![age](https://badges.renovateapi.com/packages/npm/@types%2fnode/14.17.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/@types%2fnode/14.17.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/@types%2fnode/14.17.0/compatibility-slim/10.17.60)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/@types%2fnode/14.17.0/confidence-slim/10.17.60)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: "after 9am and before 3pm" (UTC). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻️ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/cloud-profiler-nodejs). * chore(deps): update dependency sinon to v11 (#739) [![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [sinon](https://sinonjs.org/) ([source](https://togithub.com/sinonjs/sinon)) | [`^10.0.0` -> `^11.0.0`](https://renovatebot.com/diffs/npm/sinon/10.0.0/11.1.0) | [![age](https://badges.renovateapi.com/packages/npm/sinon/11.1.0/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/sinon/11.1.0/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/sinon/11.1.0/compatibility-slim/10.0.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/sinon/11.1.0/confidence-slim/10.0.0)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>sinonjs/sinon</summary> ### [`v11.1.0`](https://togithub.com/sinonjs/sinon/blob/master/CHANGELOG.md#1110--2021-05-25) [Compare Source](https://togithub.com/sinonjs/sinon/compare/v11.0.0...31be9a5d5a4762ef01cb195f29024616dfee9ce8) \================== - Add sinon.promise() implementation ([#2369](https://togithub.com/sinonjs/sinon/issues/2369)) - Set wrappedMethod on getters/setters ([#2378](https://togithub.com/sinonjs/sinon/issues/2378)) - \[Docs] Update fake-server usage & descriptions ([#2365](https://togithub.com/sinonjs/sinon/issues/2365)) - Fake docs improvement ([#2360](https://togithub.com/sinonjs/sinon/issues/2360)) - Update nise to 5.1.0 (fixed [#2318](https://togithub.com/sinonjs/sinon/issues/2318)) ### [`v11.0.0`](https://togithub.com/sinonjs/sinon/blob/master/CHANGELOG.md#1100--2021-05-24) [Compare Source](https://togithub.com/sinonjs/sinon/compare/v10.0.1...v11.0.0) \================== - Explicitly use samsam 6.0.2 with fix for [#2345](https://togithub.com/sinonjs/sinon/issues/2345) - Update most packages ([#2371](https://togithub.com/sinonjs/sinon/issues/2371)) - Update compatibility docs ([#2366](https://togithub.com/sinonjs/sinon/issues/2366)) - Update packages (includes breaking fake-timers change, see [#2352](https://togithub.com/sinonjs/sinon/issues/2352)) - Warn of potential memory leaks ([#2357](https://togithub.com/sinonjs/sinon/issues/2357)) - Fix clock test errors ### [`v10.0.1`](https://togithub.com/sinonjs/sinon/blob/master/CHANGELOG.md#1001--2021-04-08) [Compare Source](https://togithub.com/sinonjs/sinon/compare/v10.0.0...v10.0.1) \================== - Upgrade sinon components (bumps y18n to 4.0.1) - Bump y18n from 4.0.0 to 4.0.1 </details> --- ### Configuration 📅 **Schedule**: "after 9am and before 3pm" (UTC). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻️ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/cloud-profiler-nodejs). * chore: Report warning on `.github/workflows/ci.yaml` (#742) * fix: Report warning on `.github/workflows/ci.yaml` Not all files in `.github/workflows` are managed, only `ci.yaml`. Related false-positive: https://github.com/googleapis/repo-automation-bots/pull/1952#issuecomment-856142886 * fix: Report warning on `.github/workflows/ci.yaml` Not all files in `.github/workflows` are managed, only `ci.yaml`. Source-Link: https://github.com/googleapis/synthtool/commit/2430f8d90ed8a508e8422a3a7191e656d5a6bf53 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:14aaee566d6fc07716bb92da416195156e47a4777e7d1cd2bb3e28c46fe30fe2 * chore(nodejs): use cloud-rad publication process (#1112) (#744) VERSION is used in @google-cloud/cloud-rad to publish ref docs for a particular version. Pass VERSION in via Stubby or Fusion. Source-Link: https://github.com/googleapis/synthtool/commit/740366bbb9a7e0f4b77fc75dc26be1d3a376c3e0 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:bbdd52de226c00df3356cdf25460397b429ab49552becca645adbc412f6a4ed5 * build: add auto-approve to Node libraries (#1100) (#745) * build: add auto-approve to Node libraries Co-authored-by: Benjamin E. Coe <bencoe@google.com> Source-Link: https://github.com/googleapis/synthtool/commit/5cae043787729a908ed0cab28ca27baf9acee3c4 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:65aa68f2242c172345d7c1e780bced839bfdc344955d6aa460aa63b4481d93e5 * chore(nodejs): remove api-extractor dependencies (#746) * build: remove errant comma (#1113) (#747) Source-Link: https://github.com/googleapis/synthtool/commit/41ccd8cd13ec31f4fb839cf8182aea3c7156e19d Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:c9c7828c165b1985579098978877935ee52dda2b1b538087514fd24fa2443e7a * chore: clean npm cache on package installations (#748) This should avoid persistent errors in case of rare npm cache corruptions. Something like https://github.com/immerjs/immer/issues/546. * build(node): don't throw on deprecation in unit tests (#749) Fixes #1134 🦕 Removes the commit body and relative PR# from the commit message. For example, for this commit: https://github.com/googleapis/synthtool/commit/9763f20e4b7bb1091082462b2f7970e965d0d414 `post-processor-changes.txt` would contain ``` build: enable npm for php/python builds Source-Link: https://github.com/googleapis/synthtool/commit/9763f20e4b7bb1091082462b2f7970e965d0d414 ``` instead of ``` build: enable npm for php/python builds (#1133) * build: enable npm for php/python builds * update comment Source-Link: https://github.com/googleapis/synthtool/commit/9763f20e4b7bb1091082462b2f7970e965d0d414 ``` Source-Link: https://github.com/googleapis/synthtool/commit/e934b93402284f834b510ebbf421864e881dce02 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:805e2e389eafefa5ed484c30b83a7a875e6b1c7ee125d812e8b01ecc531c3fac * build(node): do not throw on deprecation (#1140) (#751) Refs https://github.com/googleapis/nodejs-service-usage/issues/22 Source-Link: https://github.com/googleapis/synthtool/commit/6d26b13debbfe3c6a6a9f9f1914c5bccf1e6fadc Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:e59b73e911585903ee6b8a1c5246e93d9e9463420f597b6eb2e4b616ee8a0fee * build: auto-approve renovate-bot PRs for minor updates (#1145) (#752) Source-Link: https://github.com/googleapis/synthtool/commit/39652e3948f455fd0b77535a0145eeec561a3706 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:41d5457ff79c3945782ab7e23bf4d617fd7bf3f2b03b6d84808010f7d2e10ca2 * chore(deps): update dependency @types/tmp to v0.2.1 (#753) [![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@types/tmp](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | [`0.2.0` -> `0.2.1`](https://renovatebot.com/diffs/npm/@types%2ftmp/0.2.0/0.2.1) | [![age](https://badges.renovateapi.com/packages/npm/@types%2ftmp/0.2.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/@types%2ftmp/0.2.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/@types%2ftmp/0.2.1/compatibility-slim/0.2.0)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/@types%2ftmp/0.2.1/confidence-slim/0.2.0)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: "after 9am and before 3pm" (UTC). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/cloud-profiler-nodejs). * fix(deps): update dependency pprof to v3.2.0 (#754) * chore: release 4.1.3 (#755) :robot: I have created a release \*beep\* \*boop\* --- ### [4.1.3](https://www.github.com/googleapis/cloud-profiler-nodejs/compare/v4.1.2...v4.1.3) (2021-07-20) ### Bug Fixes * **deps:** update dependency pprof to v3.2.0 ([#754](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/754)) ([fc48f9a](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/fc48f9aee74a3638d98086e0b404e299ae31b592)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). * build: switch to release-please release tagging (#1129) (#756) Requires https://github.com/googleapis/releasetool/pull/338 Source-Link: https://github.com/googleapis/synthtool/commit/1563597d28eca099d6411bbc29ecd09314a80746 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:06c970a44680229c1e8cefa701dbc93b80468ec4a34e6968475084e4ec1e2d7d * build: update auto-approve config for new validation (#1169) (#757) Co-authored-by: Anthonios Partheniou <partheniou@google.com> Source-Link: https://github.com/googleapis/synthtool/commit/df7fc1e3a6df4316920ab221431945cdf9aa7217 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:6245a5be4c0406d9b2f04f380d8b88ffe4655df3cdbb57626f8913e8d620f4dd * chore(nodejs): update client ref docs link in metadata (#758) * fix(build): migrate to using main branch (#760) * fix(build): migrate to using main branch * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> * build: enable release-trigger bot (#1212) (#765) Source-Link: https://github.com/googleapis/synthtool/commit/0a1b7017dec842ffe94894129c757115a8f82ae9 Post-Processor: gcr.io/repo-automation-bots/owlbot-nodejs:latest@sha256:111973c0da7608bf1e60d070e5449d48826c385a6b92a56cb9203f1725d33c3d Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> * chore: release 4.1.4 (#764) :robot: I have created a release \*beep\* \*boop\* --- ### [4.1.4](https://www.github.com/googleapis/cloud-profiler-nodejs/compare/v4.1.3...v4.1.4) (2021-09-14) ### Bug Fixes * **build:** migrate to using main branch ([#760](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/760)) ([1649406](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/164940698d7f4da0b2b6ea9e3b059e553fcfaa42)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). * chore: rename Kokoro integration test (#767) * config: switch repo-settings to main (#770) * chore: relocate owl bot post processor (#769) * chore: relocate owl bot post processor * chore: relocate owl bot post processor Co-authored-by: Benjamin E. Coe <bencoe@google.com> * build(node): run linkinator against index.html (#1227) (#772) * fix: workaround certificate expiration issue in integration tests (#773) Let's Encrypt CA certification expiration is causing integration tests to fail. This commit works around this problem by removing the problematic certificate. Context: https://letsencrypt.org/docs/dst-root-ca-x3-expiration-september-2021/ * chore: release 4.1.5 (#774) :robot: I have created a release \*beep\* \*boop\* --- ### [4.1.5](https://www.github.com/googleapis/cloud-profiler-nodejs/compare/v4.1.4...v4.1.5) (2021-10-06) ### Bug Fixes * workaround certificate expiration issue in integration tests ([#773](https://www.github.com/googleapis/cloud-profiler-nodejs/issues/773)) ([9d4908b](https://www.github.com/googleapis/cloud-profiler-nodejs/commit/9d4908b7161e0aade0915d2d88a43b0a6bfc9791)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). * chore(cloud-rad): delete api-extractor config (#775) * chore(deps): update dependency @types/node to v16 (#777) [![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [@types/node](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | [`^14.0.0` -> `^16.0.0`](https://renovatebot.com/diffs/npm/@types%2fnode/14.17.32/16.11.6) | [![age](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/compatibility-slim/14.17.32)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/confidence-slim/14.17.32)](https://docs.renovatebot.com/merge-confidence/) | --- ### Configuration 📅 **Schedule**: "after 9am and before 3pm" (UTC). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, click this checkbox. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/cloud-profiler-nodejs). * chore(deps): update dependency sinon to v12 (#778) [![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [sinon](https://sinonjs.org/) ([source](https://togithub.com/sinonjs/sinon)) | [`^11.0.0` -> `^12.0.0`](https://renovatebot.com/diffs/npm/sinon/11.1.2/12.0.1) | [![age](https://badges.renovateapi.com/packages/npm/sinon/12.0.1/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/sinon/12.0.1/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/sinon/12.0.1/compatibility-slim/11.1.2)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/sinon/12.0.1/confidence-slim/11.1.2)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>sinonjs/sinon</summary> ### [`v12.0.1`](https://togithub.com/sinonjs/sinon/blob/master/CHANGES.md#1201) [Compare Source](https://togithub.com/sinonjs/sinon/compare/v12.0.0...v12.0.1) - [`3f598221`](https://togithub.com/sinonjs/sinon/commit/3f598221045904681f2b3b3ba1df617ed5e230e3) Fix issue with npm unlink for npm version > 6 (Carl-Erik Kopseng) > 'npm unlink' would implicitly unlink the current dir > until version 7, which requires an argument - [`51417a38`](https://togithub.com/sinonjs/sinon/commit/51417a38111eeeb7cd14338bfb762cc2df487e1b) Fix bundling of cjs module ([#2412](https://togithub.com/sinonjs/sinon/issues/2412)) (Julian Gri…

idoshamun added 2 commits April 7, 2016 10:10

datastore: fix namespace isolation

0a9dd79

when using allocateId the namespace option was ignored due to typo

add ido shamun to contributors

93162f5

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 7, 2016

datastore: fix failed tests

6ed1b9b

stephenplusplus merged commit fc111c1 into googleapis:master Apr 7, 2016

snyk-bot mentioned this pull request Jan 29, 2022

[Snyk] Upgrade gaxios from 3.2.0 to 4.3.2 RebelTat/google-cloud-node#3

Open

sofisl mentioned this pull request Jan 27, 2026

migrate code from googleapis/cloud-profiler-nodejs #7044

Closed

4 tasks

This was referenced Jan 27, 2026

migrate code from googleapis/cloud-profiler-nodejs #7045

Closed

migrate code from googleapis/cloud-profiler-nodejs #7046

Closed

This was referenced Jan 27, 2026

migrate code from googleapis/cloud-profiler-nodejs #7047

Closed

migrate code from googleapis/cloud-profiler-nodejs #7048

Closed

migrate code from googleapis/cloud-profiler-nodejs #7049

Closed

migrate code from googleapis/cloud-profiler-nodejs #7051

Closed

sofisl mentioned this pull request Jan 28, 2026

migrate code from googleapis/cloud-profiler-nodejs #7052

Closed

4 tasks

sofisl mentioned this pull request Jan 29, 2026

migrate code from googleapis/cloud-profiler-nodejs #7057

Closed

4 tasks

This was referenced Jan 29, 2026

migrate code from googleapis/cloud-profiler-nodejs #7060

Closed

migrate code from googleapis/cloud-profiler-nodejs #7062

Closed

migrate code from googleapis/cloud-profiler-nodejs #7066

Closed

migrate code from googleapis/cloud-profiler-nodejs #7067

Closed

This was referenced Jan 30, 2026

migrate code from googleapis/cloud-profiler-nodejs #7068

Closed

migrate code from googleapis/cloud-profiler-nodejs #7071

Closed

GautamSharda mentioned this pull request Feb 2, 2026

migrate code from googleapis/nodejs-logging-winston #7082

Closed

GautamSharda pushed a commit that referenced this pull request Feb 2, 2026

build: enable release-trigger bot (#1212) (#626)

1cc19a0

sofisl mentioned this pull request Feb 3, 2026

migrate code from googleapis/cloud-profiler-nodejs #7087

Merged

4 tasks

sofisl pushed a commit that referenced this pull request Feb 3, 2026

build: enable release-trigger bot (#1212) (#1007)

99f75ca

sofisl pushed a commit that referenced this pull request Feb 3, 2026

chore(deps): update dependency linkinator to v5 (#1212)

78509e5

GautamSharda mentioned this pull request Feb 3, 2026

migrate code from googleapis/nodejs-logging-winston #7129

Merged

4 tasks

GautamSharda pushed a commit that referenced this pull request Feb 3, 2026

build: enable release-trigger bot (#1212) (#626)

96fdef4

sofisl mentioned this pull request Feb 4, 2026

migrate code from googleapis/nodejs-logging-bunyan #7136

Closed

4 tasks

sofisl pushed a commit that referenced this pull request Feb 4, 2026

build: enable release-trigger bot (#1212) (#562)

42cfed1

This was referenced Feb 4, 2026

migrate code from googleapis/nodejs-logging-bunyan #7137

Closed

migrate code from googleapis/nodejs-logging-bunyan #7138

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Datastore namespace fix#1212

Datastore namespace fix#1212
stephenplusplus merged 3 commits intogoogleapis:masterfrom
idoshamun:datastore-namespace-fix

idoshamun commented Apr 7, 2016

Uh oh!

stephenplusplus commented Apr 7, 2016

Uh oh!

idoshamun commented Apr 7, 2016

Uh oh!

stephenplusplus commented Apr 7, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

idoshamun commented Apr 7, 2016

Uh oh!

stephenplusplus commented Apr 7, 2016

Uh oh!

idoshamun commented Apr 7, 2016

Uh oh!

stephenplusplus commented Apr 7, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants