Updating correct path (based on repo/project name) #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ksatya77
approved these changes
Jun 1, 2018
elnyry-sam-k
pushed a commit
that referenced
this pull request
Jun 27, 2018
* added the consumer code for story #197 * added the http headers, schema validations, msg format for kafka sotry#197 * story #197 added the notification handler * added the test case * added test cases for story 197 * added test cases for story 197 * Updated a couple of files to support the functional test TODO: 1. config Kafka config 2. health end-point 3. fix functional-runner.sh * Added health endpoint and cleaned up package.json * updated .nvmrc * updates for running functional tests * fixed functional tests for story#197 * Fix merge conflicts * added integration test * made changes after the code review suggestions * unit tests * fixed some unit tests issues with notification handler tests * added unit tests * Correct path in startup command * Updating circleCI config to enable integration, functional tests and update some commands to align with template * Commit to trigger build (for PR) * Refactoring for parameterizing, updating paths, etc * Dependency issue, testing if its related to 'restoring cache' failure * Updating correct path (based on repo/project name) (#8) * Updating correct path (based on repo/project name) * Removing running five bells ledger tests * Updates regarding issue with building docker image (#10) * Updating dockerfile (#12) * Feature/#276 consume fulfill (#6) * changes for story#276 * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review-removed unwanted files * changes for story#276 after review-removed unwanted files * changes for story#276 after review-changed the kafka topic name * Feature/#275 implementSendingNotificationsToParticipant(payer,payee) (#13) * Updated the notification handler to accept fulfilment * removed space * Updated the notifications handler Updated the notifications handler with changes from the review comments * Merge from mojaloop dev merge * Fixed integration tests * fixes for issues experienced on ml-api adapter fixes for tests (#14) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * Feature/pi2demo fixes (#16) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * change return code to 200 and fixed test
mdebarros
added a commit
that referenced
this pull request
Jul 25, 2018
* added the consumer code for story #197 * added the http headers, schema validations, msg format for kafka sotry#197 * story #197 added the notification handler * added the test case * added test cases for story 197 * added test cases for story 197 * Updated a couple of files to support the functional test TODO: 1. config Kafka config 2. health end-point 3. fix functional-runner.sh * Added health endpoint and cleaned up package.json * updated .nvmrc * updates for running functional tests * fixed functional tests for story#197 * Fix merge conflicts * added integration test * made changes after the code review suggestions * unit tests * fixed some unit tests issues with notification handler tests * added unit tests * Correct path in startup command * Updating circleCI config to enable integration, functional tests and update some commands to align with template * Commit to trigger build (for PR) * Refactoring for parameterizing, updating paths, etc * Dependency issue, testing if its related to 'restoring cache' failure * Updating correct path (based on repo/project name) (#8) * Updating correct path (based on repo/project name) * Removing running five bells ledger tests * Updates regarding issue with building docker image (#10) * Updating dockerfile (#12) * Feature/#276 consume fulfill (#6) * changes for story#276 * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review-removed unwanted files * changes for story#276 after review-removed unwanted files * changes for story#276 after review-changed the kafka topic name * Feature/#275 implementSendingNotificationsToParticipant(payer,payee) (#13) * Updated the notification handler to accept fulfilment * removed space * Updated the notifications handler Updated the notifications handler with changes from the review comments * Merge from mojaloop dev merge * Fixed integration tests * fixes for issues experienced on ml-api adapter fixes for tests (#14) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * Feature/pi2demo fixes (#16) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * change return code to 200 and fixed test * updated common streaming lib for latest version of node-rdkafka (#17) * Hotfix for Issue #370 (#19) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * update producer to validate if one exists already (#18) * update producer to validate if one exists already * fixing test coverage * fixes for producer singleton not being created and fix tests * PI2 & PI3 Fix for Producer.disconnect() (#24) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * Updated Producer factory disconnect function to correctly function against singleton factory pattern. Also fixed Unit tests. * Fixed unit test coverage. * Added dependency for "rewire" to stub the internal method calls of the Producer Unit Tests
rmothilal
added a commit
that referenced
this pull request
Jun 27, 2019
* PI2 & PI3 Fix for Producer.disconnect() from Develop into Master (#25) * added the consumer code for story #197 * added the http headers, schema validations, msg format for kafka sotry#197 * story #197 added the notification handler * added the test case * added test cases for story 197 * added test cases for story 197 * Updated a couple of files to support the functional test TODO: 1. config Kafka config 2. health end-point 3. fix functional-runner.sh * Added health endpoint and cleaned up package.json * updated .nvmrc * updates for running functional tests * fixed functional tests for story#197 * Fix merge conflicts * added integration test * made changes after the code review suggestions * unit tests * fixed some unit tests issues with notification handler tests * added unit tests * Correct path in startup command * Updating circleCI config to enable integration, functional tests and update some commands to align with template * Commit to trigger build (for PR) * Refactoring for parameterizing, updating paths, etc * Dependency issue, testing if its related to 'restoring cache' failure * Updating correct path (based on repo/project name) (#8) * Updating correct path (based on repo/project name) * Removing running five bells ledger tests * Updates regarding issue with building docker image (#10) * Updating dockerfile (#12) * Feature/#276 consume fulfill (#6) * changes for story#276 * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review-removed unwanted files * changes for story#276 after review-removed unwanted files * changes for story#276 after review-changed the kafka topic name * Feature/#275 implementSendingNotificationsToParticipant(payer,payee) (#13) * Updated the notification handler to accept fulfilment * removed space * Updated the notifications handler Updated the notifications handler with changes from the review comments * Merge from mojaloop dev merge * Fixed integration tests * fixes for issues experienced on ml-api adapter fixes for tests (#14) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * Feature/pi2demo fixes (#16) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * change return code to 200 and fixed test * updated common streaming lib for latest version of node-rdkafka (#17) * Hotfix for Issue #370 (#19) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * update producer to validate if one exists already (#18) * update producer to validate if one exists already * fixing test coverage * fixes for producer singleton not being created and fix tests * PI2 & PI3 Fix for Producer.disconnect() (#24) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * Updated Producer factory disconnect function to correctly function against singleton factory pattern. Also fixed Unit tests. * Fixed unit test coverage. * Added dependency for "rewire" to stub the internal method calls of the Producer Unit Tests * updated version and dependencies made changes based on sonarqube scanning removed hard coded strings to enums * fixing code smells * fixed typo as per @lewisdaly
rmothilal
added a commit
that referenced
this pull request
Jan 27, 2020
* PI2 & PI3 Fix for Producer.disconnect() from Develop into Master (#25) * added the consumer code for story #197 * added the http headers, schema validations, msg format for kafka sotry#197 * story #197 added the notification handler * added the test case * added test cases for story 197 * added test cases for story 197 * Updated a couple of files to support the functional test TODO: 1. config Kafka config 2. health end-point 3. fix functional-runner.sh * Added health endpoint and cleaned up package.json * updated .nvmrc * updates for running functional tests * fixed functional tests for story#197 * Fix merge conflicts * added integration test * made changes after the code review suggestions * unit tests * fixed some unit tests issues with notification handler tests * added unit tests * Correct path in startup command * Updating circleCI config to enable integration, functional tests and update some commands to align with template * Commit to trigger build (for PR) * Refactoring for parameterizing, updating paths, etc * Dependency issue, testing if its related to 'restoring cache' failure * Updating correct path (based on repo/project name) (#8) * Updating correct path (based on repo/project name) * Removing running five bells ledger tests * Updates regarding issue with building docker image (#10) * Updating dockerfile (#12) * Feature/#276 consume fulfill (#6) * changes for story#276 * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review-removed unwanted files * changes for story#276 after review-removed unwanted files * changes for story#276 after review-changed the kafka topic name * Feature/#275 implementSendingNotificationsToParticipant(payer,payee) (#13) * Updated the notification handler to accept fulfilment * removed space * Updated the notifications handler Updated the notifications handler with changes from the review comments * Merge from mojaloop dev merge * Fixed integration tests * fixes for issues experienced on ml-api adapter fixes for tests (#14) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * Feature/pi2demo fixes (#16) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * change return code to 200 and fixed test * updated common streaming lib for latest version of node-rdkafka (#17) * Hotfix for Issue #370 (#19) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * update producer to validate if one exists already (#18) * update producer to validate if one exists already * fixing test coverage * fixes for producer singleton not being created and fix tests * PI2 & PI3 Fix for Producer.disconnect() (#24) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * Updated Producer factory disconnect function to correctly function against singleton factory pattern. Also fixed Unit tests. * Fixed unit test coverage. * Added dependency for "rewire" to stub the internal method calls of the Producer Unit Tests * updated dependencies for events flow Co-authored-by: Miguel de Barros <miguel@debarros.me>
rmothilal
added a commit
that referenced
this pull request
Mar 3, 2020
* PI2 & PI3 Fix for Producer.disconnect() from Develop into Master (#25) * added the consumer code for story #197 * added the http headers, schema validations, msg format for kafka sotry#197 * story #197 added the notification handler * added the test case * added test cases for story 197 * added test cases for story 197 * Updated a couple of files to support the functional test TODO: 1. config Kafka config 2. health end-point 3. fix functional-runner.sh * Added health endpoint and cleaned up package.json * updated .nvmrc * updates for running functional tests * fixed functional tests for story#197 * Fix merge conflicts * added integration test * made changes after the code review suggestions * unit tests * fixed some unit tests issues with notification handler tests * added unit tests * Correct path in startup command * Updating circleCI config to enable integration, functional tests and update some commands to align with template * Commit to trigger build (for PR) * Refactoring for parameterizing, updating paths, etc * Dependency issue, testing if its related to 'restoring cache' failure * Updating correct path (based on repo/project name) (#8) * Updating correct path (based on repo/project name) * Removing running five bells ledger tests * Updates regarding issue with building docker image (#10) * Updating dockerfile (#12) * Feature/#276 consume fulfill (#6) * changes for story#276 * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review-removed unwanted files * changes for story#276 after review-removed unwanted files * changes for story#276 after review-changed the kafka topic name * Feature/#275 implementSendingNotificationsToParticipant(payer,payee) (#13) * Updated the notification handler to accept fulfilment * removed space * Updated the notifications handler Updated the notifications handler with changes from the review comments * Merge from mojaloop dev merge * Fixed integration tests * fixes for issues experienced on ml-api adapter fixes for tests (#14) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * Feature/pi2demo fixes (#16) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * change return code to 200 and fixed test * updated common streaming lib for latest version of node-rdkafka (#17) * Hotfix for Issue #370 (#19) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * update producer to validate if one exists already (#18) * update producer to validate if one exists already * fixing test coverage * fixes for producer singleton not being created and fix tests * PI2 & PI3 Fix for Producer.disconnect() (#24) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * Updated Producer factory disconnect function to correctly function against singleton factory pattern. Also fixed Unit tests. * Fixed unit test coverage. * Added dependency for "rewire" to stub the internal method calls of the Producer Unit Tests * updated dependencies for events flow * updated to newly released version of event-sdk * dependencies updated Co-authored-by: Miguel de Barros <miguel@debarros.me>
elnyry-sam-k
pushed a commit
that referenced
this pull request
Mar 17, 2020
* PI2 & PI3 Fix for Producer.disconnect() from Develop into Master (#25) * added the consumer code for story #197 * added the http headers, schema validations, msg format for kafka sotry#197 * story #197 added the notification handler * added the test case * added test cases for story 197 * added test cases for story 197 * Updated a couple of files to support the functional test TODO: 1. config Kafka config 2. health end-point 3. fix functional-runner.sh * Added health endpoint and cleaned up package.json * updated .nvmrc * updates for running functional tests * fixed functional tests for story#197 * Fix merge conflicts * added integration test * made changes after the code review suggestions * unit tests * fixed some unit tests issues with notification handler tests * added unit tests * Correct path in startup command * Updating circleCI config to enable integration, functional tests and update some commands to align with template * Commit to trigger build (for PR) * Refactoring for parameterizing, updating paths, etc * Dependency issue, testing if its related to 'restoring cache' failure * Updating correct path (based on repo/project name) (#8) * Updating correct path (based on repo/project name) * Removing running five bells ledger tests * Updates regarding issue with building docker image (#10) * Updating dockerfile (#12) * Feature/#276 consume fulfill (#6) * changes for story#276 * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review-removed unwanted files * changes for story#276 after review-removed unwanted files * changes for story#276 after review-changed the kafka topic name * Feature/#275 implementSendingNotificationsToParticipant(payer,payee) (#13) * Updated the notification handler to accept fulfilment * removed space * Updated the notifications handler Updated the notifications handler with changes from the review comments * Merge from mojaloop dev merge * Fixed integration tests * fixes for issues experienced on ml-api adapter fixes for tests (#14) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * Feature/pi2demo fixes (#16) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * change return code to 200 and fixed test * updated common streaming lib for latest version of node-rdkafka (#17) * Hotfix for Issue #370 (#19) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * update producer to validate if one exists already (#18) * update producer to validate if one exists already * fixing test coverage * fixes for producer singleton not being created and fix tests * PI2 & PI3 Fix for Producer.disconnect() (#24) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * Updated Producer factory disconnect function to correctly function against singleton factory pattern. Also fixed Unit tests. * Fixed unit test coverage. * Added dependency for "rewire" to stub the internal method calls of the Producer Unit Tests * updated dependencies for events flow * updated to newly released version of event-sdk * Added a regex to return the fspiop-uri from the resource level no from the FSP name * added extra test Co-authored-by: Miguel de Barros <miguel@debarros.me>
rmothilal
added a commit
that referenced
this pull request
Mar 24, 2020
* PI2 & PI3 Fix for Producer.disconnect() from Develop into Master (#25) * added the consumer code for story #197 * added the http headers, schema validations, msg format for kafka sotry#197 * story #197 added the notification handler * added the test case * added test cases for story 197 * added test cases for story 197 * Updated a couple of files to support the functional test TODO: 1. config Kafka config 2. health end-point 3. fix functional-runner.sh * Added health endpoint and cleaned up package.json * updated .nvmrc * updates for running functional tests * fixed functional tests for story#197 * Fix merge conflicts * added integration test * made changes after the code review suggestions * unit tests * fixed some unit tests issues with notification handler tests * added unit tests * Correct path in startup command * Updating circleCI config to enable integration, functional tests and update some commands to align with template * Commit to trigger build (for PR) * Refactoring for parameterizing, updating paths, etc * Dependency issue, testing if its related to 'restoring cache' failure * Updating correct path (based on repo/project name) (#8) * Updating correct path (based on repo/project name) * Removing running five bells ledger tests * Updates regarding issue with building docker image (#10) * Updating dockerfile (#12) * Feature/#276 consume fulfill (#6) * changes for story#276 * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review * changes for story#276 after review-removed unwanted files * changes for story#276 after review-removed unwanted files * changes for story#276 after review-changed the kafka topic name * Feature/#275 implementSendingNotificationsToParticipant(payer,payee) (#13) * Updated the notification handler to accept fulfilment * removed space * Updated the notifications handler Updated the notifications handler with changes from the review comments * Merge from mojaloop dev merge * Fixed integration tests * fixes for issues experienced on ml-api adapter fixes for tests (#14) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * Feature/pi2demo fixes (#16) * fixes for issues experienced on ml-api adapter fixes for tests * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' (#15) * re-aligning notification logic to '2.1.3. Transfer Handler Consume (Success)' flow. Notification handler expecting action of 'fulfil' where as per SDs it should be 'commit' * fixed unit tests * change return code to 200 and fixed test * updated common streaming lib for latest version of node-rdkafka (#17) * Hotfix for Issue #370 (#19) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * update producer to validate if one exists already (#18) * update producer to validate if one exists already * fixing test coverage * fixes for producer singleton not being created and fix tests * PI2 & PI3 Fix for Producer.disconnect() (#24) * Fix for mojaloop/project#370 * Bumped up the version from 1.86.2 to 1.86.3 * Aligned version names to the end of PI2.4 (being 1.86.2) with 2.4.1 being the hotfix * Updated Producer factory disconnect function to correctly function against singleton factory pattern. Also fixed Unit tests. * Fixed unit test coverage. * Added dependency for "rewire" to stub the internal method calls of the Producer Unit Tests * updated dependencies for events flow * updated to newly released version of event-sdk * The updated event-sdk library has an issue where it makes the parentId null and this was breaking the incoming message Co-authored-by: Miguel de Barros <miguel@debarros.me>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.