Skip to content

update according to API; rustfmt - #25

Open
dancju wants to merge 1 commit into
dovahcrow:masterfrom
dancju:master
Open

update according to API; rustfmt#25
dancju wants to merge 1 commit into
dovahcrow:masterfrom
dancju:master

Conversation

@dancju

@dancju dancju commented Mar 28, 2024

Copy link
Copy Markdown

No description provided.

@dancju

dancju commented Mar 28, 2024

Copy link
Copy Markdown
Author

I just realized I shouldn't update requests.rs and definitions.rs as they are generated code. I tried to run your code generator parse swagger.ipynb but bumped into errors.

  1. assert_keys is not defined
  2. fdef in for mod, modv in fdef.modifiers.items(): is not defined.

@dovahcrow

dovahcrow commented Mar 29, 2024

Copy link
Copy Markdown
Owner

Yeah, I should make the script more runnable... I'm not using this lib recently tho...
I'll fix the script once I have time to take a look...

Meanwhile please change the generated code if that makes your code running...

@giuliorigottic

Copy link
Copy Markdown

hey @dovahcrow any update on fixing the parse swagger.ipynb?

would be happy if you could share what you got (specially in regards the missing function definitions as mentioned above) so I can take the task to make improvements to it :)

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