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

feat: separate info into a package #44

Merged
merged 30 commits into from
Sep 14, 2023
Merged

Conversation

npty
Copy link
Member

@npty npty commented Sep 12, 2023

Description

Closes AXE-1779

Separate info folder into a new package.

Distribution Contents

The distribution includes a series of ts definition files (.d.ts) and their respective map files, generated to enhance support for ts projects, despite the whole package being written in js. Here are the contents that will be published, as verified using the npm pack --dry-run command:

npm notice === Tarball Contents === 
npm notice 60B    dist/index.d.ts                   
npm notice 104B   dist/index.d.ts.map               
npm notice 800B   dist/index.js                     
npm notice 436B   dist/utils/getChainArray.d.ts     
npm notice 153B   dist/utils/getChainArray.d.ts.map 
npm notice 980B   dist/utils/getChainArray.js       
npm notice 107B   dist/utils/importNetworks.d.ts    
npm notice 154B   dist/utils/importNetworks.d.ts.map
npm notice 2.3kB  dist/utils/importNetworks.js      
npm notice 130B   dist/utils/index.d.ts             
npm notice 113B   dist/utils/index.d.ts.map         
npm notice 906B   dist/utils/index.js               
npm notice 103B   dist/utils/readJSON.d.ts          
npm notice 129B   dist/utils/readJSON.d.ts.map      
npm notice 510B   dist/utils/readJSON.js            
npm notice 27.5kB info/mainnet.json                 
npm notice 18.5kB info/stagenet.json                
npm notice 47.5kB info/testnet.json                 
npm notice 824B   package.json                      
npm notice 25B    src/index.js                      
npm notice 806B   src/utils/getChainArray.js        
npm notice 1.9kB  src/utils/importNetworks.js       
npm notice 95B    src/utils/index.js                
npm notice 332B   src/utils/readJSON.js             
npm notice 562B   tsconfig.json        

@npty npty requested a review from milapsheth September 13, 2023 09:33
@npty npty marked this pull request as ready for review September 13, 2023 09:35
@npty
Copy link
Member Author

npty commented Sep 13, 2023

@milapsheth could you verify if the cli commands still usable? 🙇‍♂️

@npty npty force-pushed the feat/separate-info-package branch from 23ae01e to 578ecca Compare September 14, 2023 06:35
axelar-chains-config/src/utils/getChainArray.js Outdated Show resolved Hide resolved
axelar-chains-config/src/utils/importNetworks.js Outdated Show resolved Hide resolved
.github/workflows/publish-chains-config.yaml Outdated Show resolved Hide resolved
evm/.example.hardhat.config.js Outdated Show resolved Hide resolved
evm/.example.hardhat.config.js Outdated Show resolved Hide resolved
@milapsheth milapsheth merged commit bc50cfe into main Sep 14, 2023
@milapsheth milapsheth deleted the feat/separate-info-package branch September 14, 2023 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants