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

Pre-release for npm #4

Merged
merged 2 commits into from
Jan 26, 2022
Merged

Pre-release for npm #4

merged 2 commits into from
Jan 26, 2022

Conversation

sheIby
Copy link
Collaborator

@sheIby sheIby commented Jan 9, 2022

  • Pre-release version 0.1.0 for node package manager
  • Testing coverage for all query functions
  • Workflow for testing and npm publish
  • Contract filtering for ArtBlocks + PBAB (mainnet/ropsten)
  • Allow errors for invalid responses
  • Refactoring

- Pre-release version 0.1.0 for node package manager
- Testing coverage for all query functions
- Workflow for testing and npm publish
- Contract filtering for ArtBlocks + PBAB (mainnet/ropsten)
- Allow errors for invalid responses
- Refactoring
@sheIby sheIby requested a review from jakerockland January 9, 2022 18:10
Copy link
Contributor

@jakerockland jakerockland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is excellent, thank you very much @sheIby

My apologies for the delay in getting this merged.

Going to follow up on this with setting CI up using Circle :) then will deploy to NPM per our chat offline.

Cheers and thank you again!

@jakerockland jakerockland merged commit 62e7868 into main Jan 26, 2022
@jakerockland jakerockland deleted the dev branch January 26, 2022 22:26
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.

2 participants