Skip to content

Conversation

@keith
Copy link
Member

@keith keith commented Sep 6, 2023

No description provided.

@keith keith force-pushed the ks/flip-incompatible_disable_objc_library_transition branch 3 times, most recently from 22a8ebd to 4dbca8a Compare September 6, 2023 21:11
@keith keith force-pushed the ks/flip-incompatible_disable_objc_library_transition branch from 4dbca8a to b3fed5c Compare September 6, 2023 22:04
@keith
Copy link
Member Author

keith commented Sep 6, 2023

@comius ok I think we're good here

@keith keith marked this pull request as ready for review September 6, 2023 22:33
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Sep 6, 2023

@Test
public void testJ2ObjcInformationExportedFromObjcLibrary() throws Exception {
setBuildLanguageOptions("--incompatible_disable_objc_library_transition=false");
Copy link
Member Author

Choose a reason for hiding this comment

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

flipping it back on these 2 was a cop out, it would be great if someone who knew the impact of j2objc changes here could review and recommend the right solution

@sgowroji sgowroji added the team-Rules-ObjC Issues for Objective-C maintainers label Sep 7, 2023
@keith
Copy link
Member Author

keith commented Sep 19, 2023

@comius bump

@keith
Copy link
Member Author

keith commented Oct 23, 2023

b4292d2

@keith keith closed this Oct 23, 2023
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-Rules-ObjC Issues for Objective-C maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants