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 vrs hash #134

Conversation

georges-berenger
Copy link

Summary: Move vrs hash to the latest version of VRS.

Differential Revision: D66797878

@facebook-github-bot
Copy link
Contributor

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

georges-berenger pushed a commit to georges-berenger/pyvrs that referenced this pull request Dec 5, 2024
Summary:
Pull Request resolved: facebookresearch#134

Move vrs hash to the latest version of VRS.

Differential Revision: D66797878
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

georges-berenger pushed a commit to georges-berenger/pyvrs that referenced this pull request Dec 5, 2024
Summary:
Pull Request resolved: facebookresearch#134

X-link: facebookresearch/vrs#168

- fmt::join has been moved in the fmt library from one header to another. We need to include both location to be sure to build in all context and in open source with older and newer versions of fmt.
- make sure the vrs hash is current
- remove some fb-only includes

Differential Revision: D66797878
@facebook-github-bot
Copy link
Contributor

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

georges-berenger pushed a commit to georges-berenger/pyvrs that referenced this pull request Dec 5, 2024
Summary:
Pull Request resolved: facebookresearch#134

X-link: facebookresearch/vrs#168

- fmt::join has been moved in the fmt library from one header to another. We need to include both location to be sure to build in all context and in open source with older and newer versions of fmt.
- make sure the vrs hash is current
- remove some fb-only includes
- remove annotations from oss

Reviewed By: finik

Differential Revision: D66797878
@facebook-github-bot
Copy link
Contributor

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

Summary:
Pull Request resolved: facebookresearch#134

X-link: facebookresearch/vrs#168

- fmt::join has been moved in the fmt library from one header to another. We need to include both location to be sure to build in all context and in open source with older and newer versions of fmt.
- make sure the vrs hash is current
- remove some fb-only includes
- remove annotations from oss
- fix license linter

Reviewed By: finik

Differential Revision: D66797878
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in bffa874.

facebook-github-bot pushed a commit to facebookresearch/vrs that referenced this pull request Dec 5, 2024
Summary:
X-link: facebookresearch/pyvrs#134

Pull Request resolved: #168

- fmt::join has been moved in the fmt library from one header to another. We need to include both location to be sure to build in all context and in open source with older and newer versions of fmt.
- make sure the vrs hash is current
- remove some fb-only includes
- remove annotations from oss
- fix license linter

Reviewed By: finik

Differential Revision: D66797878

fbshipit-source-id: 373948222270d7ccde7779dd9b630a09a9fa56de
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants