Skip to content

Rename _InstructionCounter #632

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Sep 18, 2023
Merged

Conversation

bnbarham
Copy link
Contributor

In the unified build there ends up being two of these, one in swift-syntax and one in swift-format. Rename so that the target is unique.

In the unified build there ends up being two of these, one in
swift-syntax and one in swift-format. Rename so that the target is
unique.
Copy link
Contributor

@glessard glessard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was going to suggest exactly this name.

@bnbarham
Copy link
Contributor Author

I'm not really sure why cross PR testing didn't catch this 🤔

@glessard glessard merged commit 8488a21 into swiftlang:main Sep 18, 2023
@bnbarham bnbarham deleted the unified-pm-broken branch September 18, 2023 23:32
@allevato
Copy link
Member

@bnbarham
Copy link
Contributor Author

It does :(

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants