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

ace_mode documentation URL out of date? #6342

Closed
ryandesign opened this issue Mar 27, 2023 · 1 comment · Fixed by #6344
Closed

ace_mode documentation URL out of date? #6342

ryandesign opened this issue Mar 27, 2023 · 1 comment · Fixed by #6344
Labels

Comments

@ryandesign
Copy link
Contributor

I'm attempting to follow the documentation for adding a new language. It says:

https://github.com/github/linguist/blob/5c40ebba637fc2ef49d24148ab1b5844fe026399/lib/linguist/languages.yml#L9-L11

That URL http://git.io/3XO_Cg goes to https://github.com/ajaxorg/ace/tree/master/lib/ace/mode and I don't see anything in that directory listing that helps me understand the valid values for ace_mode.

Are the filenames you were wanting me to see the ones that were moved in ajaxorg/ace@bd8581c to https://github.com/ajaxorg/ace/tree/master/src/mode? If so please update the URL in the documentation.

@lildude
Copy link
Member

lildude commented Mar 27, 2023

Are the filenames you were wanting me to see the ones that were moved in ajaxorg/ace@bd8581c to https://github.com/ajaxorg/ace/tree/master/src/mode? If so please update the URL in the documentation.

Yes. Fix coming in #6344

@Alhadis Alhadis changed the title ace_mode documentation URL out of date? ace_mode documentation URL out of date? Mar 27, 2023
@github-linguist github-linguist locked as resolved and limited conversation to collaborators Jun 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants