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 2to3 broken symlink #218

Merged
merged 1 commit into from
Sep 11, 2024
Merged

Fix 2to3 broken symlink #218

merged 1 commit into from
Sep 11, 2024

Conversation

NouemanKHAL
Copy link
Member

@NouemanKHAL NouemanKHAL commented Sep 11, 2024

What does this PR do?

Fix a broken symlink 2to3 pointing to an old version of python file.

lrwxrwxrwx  1 vcap vcap         8 Sep  7 17:48 2to3 -> 2to3-3.8
-rwxr-xr-x  1 vcap vcap       119 Sep  7 17:48 2to3-3.11*

Verification Process

Added a test during the release process to scan for broken symlinks.

Additional Notes

https://datadoghq.atlassian.net/browse/PLINT-502

Release Notes

Review checklist (to be filled by reviewers)

  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have one changelog/ label attached. If applicable it should have the backward-incompatible label attached.
  • PR should not have do-not-merge/ label attached.
  • If Applicable, issue must have kind/ and severity/ labels attached at least.

@NouemanKHAL NouemanKHAL added the changelog/Fixed Fixed features results into a bug fix version bump label Sep 11, 2024
@NouemanKHAL NouemanKHAL marked this pull request as ready for review September 11, 2024 11:54
@NouemanKHAL NouemanKHAL requested a review from a team September 11, 2024 11:54
@NouemanKHAL NouemanKHAL merged commit 9c42e39 into master Sep 11, 2024
2 checks passed
@NouemanKHAL NouemanKHAL deleted the noueman/fix-broken-symlink branch September 11, 2024 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/Fixed Fixed features results into a bug fix version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants