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

Fixes #5446 Do not convert type parameter to comment type #5960

Closed
wants to merge 5 commits into from

Conversation

kogai
Copy link

@kogai kogai commented Mar 10, 2019

Fixes #5446

When once comment type of Flow occurred at the code, unintentional converting from type parameter like A<B> to A/*:: <B> */ applied.

As commented at here, due to lack of information whether the current node declared internal comment type or not, differences this PR proposed to do not fully ideal.

  • I’ve added tests to confirm my change works.
  • (If changing the API or CLI) I’ve documented the changes I’ve made (in the docs/ directory)
  • (If the change is user-facing) I’ve added my changes to the CHANGELOG.unreleased.md file following the template.
  • I’ve read the contributing guidelines.

Try the playground for this PR

@kogai
Copy link
Author

kogai commented Apr 23, 2019

Could somebody review or should I close it includes non-ideal solution ❓

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Can we add original example? Also please add to changelog changes, thanks for PR

@fisker
Copy link
Member

fisker commented May 22, 2020

Did I break it?

@alexander-akait
Copy link
Member

@fisker Go ahead

Base automatically changed from master to main January 23, 2021 17:13
@thorn0
Copy link
Member

thorn0 commented Oct 24, 2022

superseded by #13687

@thorn0 thorn0 closed this Oct 24, 2022
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.

Comment types of Flow makes Prettier confused
4 participants