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

shell lexer: add Alpine Linux APKBUILD (simple two line patch) #1099

Merged
merged 1 commit into from
May 29, 2019

Conversation

ollieparanoid
Copy link
Contributor

Properly highlight Alpine's package build recipes. Done in the same way as 5dca906 for Arch Linux.

Downstream feature request:
https://gitlab.com/gitlab-org/gitlab-ce/issues/56579

@ollieparanoid
Copy link
Contributor Author

CI failure is not related to the patch.

@dblessing: it would be great if you could take a look at this simple two line change. The feature has >20 upvotes in the gitlab-ce issue tracker.

@ollieparanoid ollieparanoid changed the title shell lexer: add Alpine Linux APKBUILD shell lexer: add Alpine Linux APKBUILD (simple two line patch) May 21, 2019
@ollieparanoid
Copy link
Contributor Author

FWIW, I'm still highly invested in getting this merged. Thank you in advance to whoever steps up to maintain rouge in the future, and is reading this. Your work is appreciated very much.

@pyrmont
Copy link
Contributor

pyrmont commented May 28, 2019

@ollieparanoid Sorry it's taken a while to get back to you on this :( Can you do a force push and kick off a new CI testing cycle now that we've got that fixed?

Something like:

$ git commit --amend
$ git push --force

should work.

@pyrmont pyrmont added the author-action The PR has been reviewed but action by the author is needed label May 28, 2019
Properly highlight Alpine's package build recipes. Done in the same way
as 5dca906 for Arch Linux.
@ollieparanoid
Copy link
Contributor Author

@pyrmont: sure, done! Happy to see activity here :)

@pyrmont pyrmont merged commit 273de5e into rouge-ruby:master May 29, 2019
@pyrmont pyrmont removed the author-action The PR has been reviewed but action by the author is needed label May 29, 2019
@pyrmont
Copy link
Contributor

pyrmont commented May 29, 2019

No worries :) Still discussing with the other maintainers when to release a new build of the gem but hopefully it'll be soon!

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.

2 participants