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

build: continue repo setup #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

janniks
Copy link
Collaborator

@janniks janniks commented Sep 27, 2024

  • added lerna
  • added github actions

I tried to make semantic-release work, but I'm not sure I like it -- lmk if I should switch back to it

package.json Show resolved Hide resolved
src/schema.ts Outdated
Comment on lines 360 to 365
'/v2/constant_val/{contract_address}/{contract_name}/{constant_name}': {
parameters: {
query?: never;
header?: never;
path?: never;
cookie?: never;
Copy link
Member

Choose a reason for hiding this comment

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

Upstream openapi schema is so limited/broken. Thoughts on using an improved local copy for now which we PR to upstream?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure we can maintain something. Probably easier than trying to regularly PR into core

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