Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lint warning fixes #318

Merged
merged 1 commit into from
Oct 14, 2021
Merged

Lint warning fixes #318

merged 1 commit into from
Oct 14, 2021

Conversation

A-5ingh
Copy link
Contributor

@A-5ingh A-5ingh commented Oct 13, 2021

Fixes #305

  • Fixed lint warning in chaincode

  • Fixed linting warnings in typescript_app

  • Fixed lint warning due to prettier auto save

@A-5ingh A-5ingh changed the title Lint warning fixes (#1) Lint warning fixes Oct 13, 2021
@lgtm-com
Copy link

lgtm-com bot commented Oct 14, 2021

This pull request introduces 1 alert when merging c251169 into d652dfd - view on LGTM.com

new alerts:

  • 1 for Syntax error

@Zzocker
Copy link
Member

Zzocker commented Oct 14, 2021

@A-5ingh , Please Squash your commit into one.use git rebase -i <your_first_commit>

* Fixed lint warning in chaincode

* Fixed linting warnings in typescript_app

* Fixed lint warning due to prettier auto save

Signed-off-by: Amarbir Singh <amarbir1800@gmail.com>

Lint warning fixes (#2)

* Fixed lint warning in chaincode

* Fixed linting warnings in typescript_app

* Fixed lint warning due to prettier auto save

* Fixed no-return-type lint warnings

Signed-off-by: Amarbir Singh <amarbir1800@gmail.com>

Update emissionsRecordContract.ts

Bug fix while merge request

Signed-off-by: Amarbir Singh <amarbir1800@gmail.com>

Typo fix

Signed-off-by: Amarbir Singh <amarbir1800@gmail.com>

Fixed two more typos

Signed-off-by: Amarbir Singh <amarbir1800@gmail.com>
@udosson udosson merged commit afb687e into hyperledger-labs:main Oct 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix lint warnings in typescript_app and chaincode directory
3 participants