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

README.md: tweak wording #104

Open
wants to merge 1 commit into
base: release
Choose a base branch
from
Open

README.md: tweak wording #104

wants to merge 1 commit into from

Conversation

SamB
Copy link

@SamB SamB commented Jun 28, 2022

There's no need to use "irregardless" when we have "regardless".

(I'm actually just trying to trigger a build failure without changing any of the moving parts, so I just made this tweak because it was bugging me anyway.)

There's no need to use "irregardless" when we have "regardless".
@SamB
Copy link
Author

SamB commented Jun 28, 2022

Oh. Is there no automated CI on PRs here?

@ice-phoenix
Copy link
Contributor

Hey mate, thanks for your interest in the Kotlin spec =)

Oh. Is there no automated CI on PRs here?

Nope, we never had the need for them previously.

I'm actually just trying to trigger a build failure without changing any of the moving parts

What's your endgame here, i.e., what exactly are you trying to achieve?

@SamB
Copy link
Author

SamB commented Jun 29, 2022

I was just trying to let you guys know that antlr 4.10 broke the runtime ABI, and that seems to have broken the spec build process. A failed CI build seemed like it would be a good way to do that.

@ice-phoenix
Copy link
Contributor

Yeah, we know about that break and the workaround fix is in the pipeline together with the changes for the 1.7.0 spec release. In the meantime I recorded your suggestion as #105. Thanks!

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