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

Adopt pure ruby DSL implementation for JRuby #8997

Closed
wants to merge 17 commits into from

Conversation

JasonLunn
Copy link
Contributor

@JasonLunn JasonLunn commented Sep 18, 2021

Also squashes all current JRuby unit test failures and resets expectations about known conformance test failures to be the super set of Java + Ruby known failures.

Includes all changes separately posted under #8954

Since this is the rake target used by kokoro, this will close the gem release gap for JRuby

Should help protocolbuffers#7923 by making sure JRuby is built and released as part of the normal cadence.
…olves most test failures under JRuby 9.2.11.1
…olves most test failures under JRuby 9.2.11.1
…tests

# Conflicts:
#	ruby/src/main/java/com/google/protobuf/jruby/RubyDescriptorPool.java
…olves all unit test failures under JRuby 9.2.11.1
chrusty and others added 6 commits September 22, 2021 18:15
* deps: add j2objc annotations

@pzd this will be needed on the next sync with google3
…xed, and signed scalar types as supported map keys.

Reset JRuby conformance test expectations to align with the superset of expected failures from both Ruby and Java.

Includes minor cleanups to fix typos, to remove unused variables, unused private methods, and unused imports, and improve formatting.
@google-cla
Copy link

google-cla bot commented Oct 1, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla google-cla bot added cla: no and removed cla: yes labels Oct 1, 2021
@google-cla
Copy link

google-cla bot commented Oct 1, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@google-cla
Copy link

google-cla bot commented Oct 1, 2021

All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter.

We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only @googlebot I consent. in this pull request.

Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the cla label to yes (if enabled on your project).

ℹ️ Googlers: Go here for more info.

@JasonLunn
Copy link
Contributor Author

Closing in favor of #9047

@JasonLunn JasonLunn closed this Oct 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants