Skip to content

[monarch] don't spawn AutoReloadActor if auto_reload=False #592

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

Closed
wants to merge 1 commit into from

Conversation

suo
Copy link
Contributor

@suo suo commented Jul 20, 2025

Stack from ghstack (oldest at bottom):

we don't always want to set the import audit hooks

Differential Revision: D78636824

NOTE FOR REVIEWERS: This PR has internal Meta-specific changes or comments, please review them on Phabricator!

we don't always want to set the import audit hooks

Differential Revision: [D78636824](https://our.internmc.facebook.com/intern/diff/D78636824/)

**NOTE FOR REVIEWERS**: This PR has internal Meta-specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D78636824/)!

[ghstack-poisoned]
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 20, 2025
suo added a commit that referenced this pull request Jul 20, 2025
we don't always want to set the import audit hooks

Differential Revision: [D78636824](https://our.internmc.facebook.com/intern/diff/D78636824/)

**NOTE FOR REVIEWERS**: This PR has internal Meta-specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D78636824/)!

ghstack-source-id: 297393997
Pull Request resolved: #592
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D78636824

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 0aae80c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants