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

Add GitHub build server #290

Merged
merged 3 commits into from
Oct 30, 2021
Merged

Conversation

smoogipoo
Copy link
Contributor

@smoogipoo smoogipoo commented Jun 15, 2021

I hope this is fine without tests - it's an MVP implementation using console-based annotations.

Seen working here: https://github.com/smoogipoo/osu-framework/pull/28/checks (and check the files listing for annotations).

@laedit
Copy link
Owner

laedit commented Jun 15, 2021

Thanks for your contribution!
That looks great, I will review it asap.

@smoogipoo
Copy link
Contributor Author

As mentioned in ppy/osu-framework#4514 (comment), I'll need to take a look into this again and probably use the API, since we can only have a limited number of annotations and there's no display other than:
image
Which doesn't help much...

@smoogipoo
Copy link
Contributor Author

Maybe it's fine to get this in as-is for now? From experience having used my fork with this PR, the output has been adequate. I haven't yet been able to find the time to implement API annotations :(

@laedit
Copy link
Owner

laedit commented Oct 30, 2021

Sounds good to me, better have something than nothing at all 😄

@laedit laedit merged commit 95a178a into laedit:master Oct 30, 2021
laedit added a commit that referenced this pull request Oct 30, 2021
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