Skip to content

Fix Swift and JavaScript code imports. #208

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

Merged
merged 2 commits into from
Jul 3, 2018
Merged

Fix Swift and JavaScript code imports. #208

merged 2 commits into from
Jul 3, 2018

Conversation

june128
Copy link
Member

@june128 june128 commented Jul 3, 2018

No description provided.

@june128 june128 added the Small Changes This changes a little error (regarding linking or similiar) and doesn't fit another category. label Jul 3, 2018
@leios
Copy link
Member

leios commented Jul 3, 2018

Ah, man. I was just about to do that. We need to figure out what the appropriate label is for swift. It should be in this file: https://github.com/algorithm-archivists/highlight.js/blob/master/src/languages/swift.js (I think)

@june128
Copy link
Member Author

june128 commented Jul 3, 2018

Hmm yeah, I try to solve it... Sorry for your PR!

@leios
Copy link
Member

leios commented Jul 3, 2018

As a note: once we get these tags fixed, I was planning on pushing to the github pages repo. No rush, though. I cannot even do that until I get home tonight anyway.

Copy link
Member

@leios leios left a comment

Choose a reason for hiding this comment

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

These need to be updated, so it's a good change!

@june128 june128 merged commit e19f00e into algorithm-archivists:master Jul 3, 2018
@june128 june128 deleted the fixCodeImportAndHighlightingPR branch July 3, 2018 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Small Changes This changes a little error (regarding linking or similiar) and doesn't fit another category.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants