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

fix: set dbx volume var once #685

Merged
merged 1 commit into from
Dec 4, 2024
Merged

Conversation

jayengee
Copy link
Contributor

@jayengee jayengee commented Dec 4, 2024

Summary

Previous PR re-defined an attribute, which is not allowed by downstream linters given that its bad practice

Test Plan

Say unittests, or list out steps to verify changes.

References

(Optional) Additional links to provide more context.

@jayengee jayengee requested a review from a team December 4, 2024 22:09
@jayengee jayengee marked this pull request as ready for review December 4, 2024 22:09
@jayengee jayengee requested a review from a team as a code owner December 4, 2024 22:09
@jayengee jayengee requested a review from beroy December 4, 2024 22:09
@jayengee jayengee merged commit 5e76ad3 into main Dec 4, 2024
109 checks passed
@jayengee jayengee deleted the jng/fix-dbx-volume-permissions-pt2 branch December 4, 2024 22:22
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