-
Notifications
You must be signed in to change notification settings - Fork 6
feat: added art/grid API endpoint #46
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
base: main
Are you sure you want to change the base?
Conversation
I wasn't sure if I should put it as draft. But I'll do it when back to pc as GitHub mobile doesn't seem to have the features |
Thanks @InputUsername =) |
bb09a91
to
435f00d
Compare
Sorry for the absence, I was sick all weekend (yay). It seems like |
@InputUsername It also seems the PR I merged yesterday wasn't rebased, so CI didn't warn us about the MSRV job failing. We need to have a discussion - in a separate issue maybe? -- about going forward with MSRV. |
The previous PR has been revised tho? And the msrv check passed. Probably an update happened between the checks Edit: that new version is freshly out: |
I've made #47 to continue the discussion. |
800bed9
to
3abddec
Compare
@RustyNova016 You'll need to |
Head branch was pushed to by a user without write access
3abddec
to
78b955d
Compare
78b955d
to
b02278e
Compare
Added this api endpoint. Since the documentation is quite vague about it, I expended some comments, and added an exemple (it doesn't actually run. It only try to compile)
#31 need to pass first, as I based it on this branch.