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 local override of enum prefix-with-name #1006

Merged
merged 1 commit into from
Oct 27, 2024

Conversation

jsgf
Copy link
Contributor

@jsgf jsgf commented Oct 4, 2024

Fixes #808 using @emilio's suggestion in #808 (comment)

facebook-github-bot pushed a commit to facebookexperimental/rust-shed that referenced this pull request Oct 4, 2024
Summary: [Fix an upstream bug](mozilla/cbindgen#1006) and [add ability to put arbitrary prefix on field names](mozilla/cbindgen#1007).

Reviewed By: jasonwhite

Differential Revision: D63878067

fbshipit-source-id: 16a35ed1fa2adc4c9211fc6f6332f0be7dbf4f56
facebook-github-bot pushed a commit to facebook/sapling that referenced this pull request Oct 4, 2024
Summary: [Fix an upstream bug](mozilla/cbindgen#1006) and [add ability to put arbitrary prefix on field names](mozilla/cbindgen#1007).

Reviewed By: jasonwhite

Differential Revision: D63878067

fbshipit-source-id: 16a35ed1fa2adc4c9211fc6f6332f0be7dbf4f56
facebook-github-bot pushed a commit to facebook/hhvm that referenced this pull request Oct 4, 2024
Summary: [Fix an upstream bug](mozilla/cbindgen#1006) and [add ability to put arbitrary prefix on field names](mozilla/cbindgen#1007).

Reviewed By: jasonwhite

Differential Revision: D63878067

fbshipit-source-id: 16a35ed1fa2adc4c9211fc6f6332f0be7dbf4f56
@emilio
Copy link
Collaborator

emilio commented Oct 27, 2024

Looks good tho a test would be nice. Given it's trivial and I don't have the cycles to do it myself right now, and I've already taken too long to get to this, this is fine.

@emilio emilio added this pull request to the merge queue Oct 27, 2024
Merged via the queue into mozilla:master with commit 6584efa Oct 27, 2024
2 checks passed
@jsgf jsgf deleted the fix-issue-808 branch October 28, 2024 19:08
facebook-github-bot pushed a commit to facebook/dotslash that referenced this pull request Oct 29, 2024
Summary:
PRs mozilla/cbindgen#1006
mozilla/cbindgen#1007 were accepted, so update patch to
upstream git repo until next release is cut.

Also bump `syn` to match cbindgen.

Reviewed By: dtolnay

Differential Revision: D65079945

fbshipit-source-id: 5b9f0467150d7cf81f5a9df0bb4d0c1309e67e1b
facebook-github-bot pushed a commit to facebookexperimental/rust-shed that referenced this pull request Oct 29, 2024
Summary:
PRs mozilla/cbindgen#1006
mozilla/cbindgen#1007 were accepted, so update patch to
upstream git repo until next release is cut.

Also bump `syn` to match cbindgen.

Reviewed By: dtolnay

Differential Revision: D65079945

fbshipit-source-id: 5b9f0467150d7cf81f5a9df0bb4d0c1309e67e1b
facebook-github-bot pushed a commit to facebook/sapling that referenced this pull request Oct 29, 2024
Summary:
PRs mozilla/cbindgen#1006
mozilla/cbindgen#1007 were accepted, so update patch to
upstream git repo until next release is cut.

Also bump `syn` to match cbindgen.

Reviewed By: dtolnay

Differential Revision: D65079945

fbshipit-source-id: 5b9f0467150d7cf81f5a9df0bb4d0c1309e67e1b
facebook-github-bot pushed a commit to facebookincubator/reindeer that referenced this pull request Oct 29, 2024
Summary:
PRs mozilla/cbindgen#1006
mozilla/cbindgen#1007 were accepted, so update patch to
upstream git repo until next release is cut.

Also bump `syn` to match cbindgen.

Reviewed By: dtolnay

Differential Revision: D65079945

fbshipit-source-id: 5b9f0467150d7cf81f5a9df0bb4d0c1309e67e1b
facebook-github-bot pushed a commit to facebook/hhvm that referenced this pull request Oct 29, 2024
Summary:
PRs mozilla/cbindgen#1006
mozilla/cbindgen#1007 were accepted, so update patch to
upstream git repo until next release is cut.

Also bump `syn` to match cbindgen.

Reviewed By: dtolnay

Differential Revision: D65079945

fbshipit-source-id: 5b9f0467150d7cf81f5a9df0bb4d0c1309e67e1b
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.

cbindgen:prefix-with-name=false comment does not override toml value enum.prefix_with_name=true
2 participants