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

only check version once #912

Closed
wants to merge 1 commit into from
Closed

only check version once #912

wants to merge 1 commit into from

Conversation

Janther
Copy link
Contributor

@Janther Janther commented Aug 21, 2023

isPrettier2 should be a constant, not a function.

@Janther Janther added enhancement New feature or request refactor reorganising the code without changes and removed enhancement New feature or request labels Oct 1, 2023
Janther added a commit that referenced this pull request Oct 31, 2023
…o a constant checked at the loading of the module
@Janther
Copy link
Contributor Author

Janther commented Oct 31, 2023

If #941 is merged, this one should be ignored.

Janther added a commit that referenced this pull request Nov 7, 2023
…tation (#941)

* Proposal for avoiding separation between function definitions without an implementation

* compatibility with Prettier 2

* refactor backward compatibility utils

* moving isPrettier2 to the backwards compatibility module

* reorganising the logic for clarity and to avoid having to check for extra lines added by mistake at the end

* to avoid conflicts with #912 we changed isPrettier2 from a function to a constant checked at the loading of the module

* Adding documentation

* only check for leading comments
@Janther Janther closed this Nov 8, 2023
@Janther
Copy link
Contributor Author

Janther commented Nov 8, 2023

closing as this behaviour was in #941

@Janther Janther deleted the small-detail branch November 8, 2023 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor reorganising the code without changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant