Skip to content
This repository was archived by the owner on May 8, 2023. It is now read-only.

fix: makes types and abi available to publish#11

Open
jurajpiar wants to merge 1 commit intorsksmart:masterfrom
jurajpiar:hotfix/includetypes
Open

fix: makes types and abi available to publish#11
jurajpiar wants to merge 1 commit intorsksmart:masterfrom
jurajpiar:hotfix/includetypes

Conversation

@jurajpiar
Copy link
Copy Markdown
Member

@jurajpiar jurajpiar commented Dec 31, 2021

Previously, /types and /ERC677Data.json have been added to the files property in package.json so that these are present in the package after it is installed in a project. This has been reflected in the documentation, but presumably mistakenly not removed from .gitignore, thus never pushed to the repo itself.

This PR changes that, whilst removing the type generation from the installation process, as these will be already present.

Resolves #10

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 31, 2021

Pull Request Test Coverage Report for Build 1641001813

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 90.909%

Totals Coverage Status
Change from base Build 764899969: 0.0%
Covered Lines: 9
Relevant Lines: 9

💛 - Coveralls

@jurajpiar
Copy link
Copy Markdown
Member Author

I'm not entirely clear on why the slither scan is failing @ilanolkies, but maybe @luisfontes19 could shed some light on this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Installation via npm does not compile

2 participants