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

Rename Delphi to Pascal #677

Merged
merged 2 commits into from Sep 9, 2013
Merged

Rename Delphi to Pascal #677

merged 2 commits into from Sep 9, 2013

Conversation

haileys
Copy link
Contributor

@haileys haileys commented Sep 6, 2013

Certain repositories (like https://github.com/graemeg/freepascal) are being shown as being written in Delphi instead of Pascal.

While Delphi is technically a different language to Pascal, I think it's still appropriate to call Delphi code Pascal, even if it may not be appropriate to call non-Delphi Pascal code (like a self-hosting Pascal compiler) Delphi. (does that make sense?)

cc @tnm

tnm added a commit that referenced this pull request Sep 9, 2013
@tnm tnm merged commit c4b21f5 into master Sep 9, 2013
@haileys haileys deleted the rename-delphi-to-pascal branch September 9, 2013 06:14
@jamiei
Copy link

jamiei commented Oct 2, 2013

This needs rolling back. Pure Delphi projects are now detected as "Other".

The removal of the samples in lib/linguist/samples.json might be the cause. It might be reasonable to call both Pascal with the correct detection in place although, I think having both Delphi and Pascal would be better, even if the split isn't possible to get correct in small repos. Delphi is easier to identify than Pascal and will not compile (without the Delphi compatibility flag) under something like Freepascal.

@Faq
Copy link

Faq commented Jan 23, 2014

And all delphi projects show as pascal now...
Example:
https://github.com/Faq/TXTGenerator
https://github.com/Gurux/Gurux.DLMS.Delphi

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.

None yet

4 participants