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

state of the project? #77

Closed
7 of 12 tasks
splitbrain opened this issue Mar 12, 2017 · 6 comments
Closed
7 of 12 tasks

state of the project? #77

splitbrain opened this issue Mar 12, 2017 · 6 comments

Comments

@splitbrain
Copy link
Contributor

splitbrain commented Mar 12, 2017

Is this project still actively maintained? We make extensive use of this library in DokuWiki and I'd like to have a place where to point people to with language updates.

Would you consider adding more maintainers to get this at least into a state where minimal changes (language fixes, PHP compatibility updates, etc.) get applied? I'd be happy to help with that.

Here are the things I think should be done:

Again, I'm happy to do this if I get the feedback that it would be welcome and be merged quickly.

@cweiske
Copy link
Member

cweiske commented Mar 12, 2017

A namespace will no be introduced since it will break backwards compatibility. Use geshi-1.1 for that.

What I would really miss is unit tests that tell me if a patch breaks things :/

@splitbrain
Copy link
Contributor Author

Yeah. Unit tests would be awesome, but I don't know enough about the GeSHi internals to do that. However I thought about two things that would be relatively easy:

  1. dogfood the geshi source through geshi and simply check that a few keywords are in the resulting output -- this would ensure at least the PHP compatibility and catch possible syntax errors.
  2. use the checklang.php mechanism in a unit test

When both are in place it could be set up at travis and you could at least see if new PRs violate those basic things already. Eg. I had to fix most of the language additions becasue checklang found some issues

@cweiske
Copy link
Member

cweiske commented Mar 12, 2017

unit tests would be too much,yes. I thought about having a set of source test files and their expected output, which gets compared by phpunit.

But having checklang in travis would also help, yes.

@splitbrain
Copy link
Contributor Author

@cweiske I started working on unit testing as outlined above. Can you have a look at my questions here: splitbrain-forks#1

@splitbrain
Copy link
Contributor Author

I converted the geshi-doc.html document to markdown, manually fixed stuff and added the contents (mostly) to the wiki. I left out stuff that very obviously needs rewriting. A new table of contents linking to the various documents should be created on the Home page. And of course all the pages need some work, but now the community could step in.

@splitbrain
Copy link
Contributor Author

I'm closing this in favor of #79 #81 and #82

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

No branches or pull requests

2 participants