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

Add Embarcadero formatter #23

Merged
merged 1 commit into from
Sep 1, 2020
Merged

Add Embarcadero formatter #23

merged 1 commit into from
Sep 1, 2020

Conversation

AThePeanut4
Copy link
Contributor

Hi, this is just my quick edit to add basic support for the Embarcadero formatter.exe.

@alefragnani
Copy link
Owner

Hi @AThePeanut4 ,

I'll try to test it next week, and if everything works fine, it should be available in the next release.

Just out of curiosity, which Delphi release you did use test it? I mean, I don't think it should make any difference (their formatter must be the same for a long time), unless they added updated on Rio (inline var), or Sidney (LSP)

Thank you

@alefragnani alefragnani self-requested a review August 13, 2020 02:41
@alefragnani alefragnani self-assigned this Aug 13, 2020
@alefragnani alefragnani added the enhancement New feature or request label Aug 13, 2020
@alefragnani alefragnani added this to the 2.4.0 milestone Aug 13, 2020
@alefragnani alefragnani linked an issue Aug 13, 2020 that may be closed by this pull request
@AThePeanut4
Copy link
Contributor Author

I'm using Sydney, but when I ran the formatter.exe with no args the help dialog said 'Copyright 2010' so I don't think it's changed since then.

@alefragnani alefragnani merged commit 82f2909 into alefragnani:master Sep 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Embarcadero Formatter
2 participants