Skip to content
This repository has been archived by the owner on Nov 20, 2024. It is now read-only.

mark avoid_dynamic_calls stable #2853

Merged
merged 1 commit into from
Aug 6, 2021
Merged

mark avoid_dynamic_calls stable #2853

merged 1 commit into from
Aug 6, 2021

Conversation

pq
Copy link
Member

@pq pq commented Aug 4, 2021 •
edited
Loading

@google-cla google-cla bot added the cla: yes label Aug 4, 2021
@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.978% when pulling e99668a on avoid_dynamic_calls_stable into 73ba812 on master.

bwilkerson
bwilkerson approved these changes Aug 6, 2021
View reviewed changes
Copy link
Member

@bwilkerson bwilkerson left a comment

Choose a reason for hiding this comment

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

As far as I know this should be fine. I don't remember why it was marked as experimental, though, so there might be a reason I don't know for not doing this.

@pq
Copy link
Member Author

pq commented Aug 6, 2021

It just needed some baking in. I think we're well past that now.

Thanks!

@pq pq merged commit 44c04a9 into master Aug 6, 2021
@pq pq deleted the avoid_dynamic_calls_stable branch August 6, 2021 23:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bwilkerson bwilkerson bwilkerson approved these changes

Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Graduate avoid_dynamic_calls from experimental
3 participants
@pq @coveralls @bwilkerson