Skip to content

CI: add reporting endpoint - #243

Merged
ev-br merged 1 commit into
data-apis:mainfrom
ev-br:reporting
Aug 31, 2026
Merged

CI: add reporting endpoint#243
ev-br merged 1 commit into
data-apis:mainfrom
ev-br:reporting

Conversation

@ev-br

@ev-br ev-br commented Aug 31, 2026

Copy link
Copy Markdown
Member

No description provided.

@ev-br
ev-br merged commit ce01e34 into data-apis:main Aug 31, 2026
24 of 25 checks passed
@lucascolley

Copy link
Copy Markdown
Member

nit, I think the LLM can calm down on the number of one line comments

Comment on lines +75 to +76
# Install Python:
- name: 'Install Python'

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this is a bit ridiculous 😭

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You're free to strip the comments.

@ev-br

ev-br commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

Yeah, let's bring this to the attention of the author of the original version--- I'm pretty sure most users would do pretty much what I did here, copy-paste and add absolutely minimal changes. cc @kgryte

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